[#19075] - Ensure catid to be an integer array in search archived articles
- Fixed in Code Base
- 18 Dec 2017
- Medium
- Build: staging
- # 19075
- Diff
- Quy:patch-13
User tests: Successful: Unsuccessful:
Summary of Changes
In searching archived articles, catid is expected to an array, however, it can be changed in the URL to be non-array which results in warnings. This PR ensures that catid be an (integer) array.
Testing Instructions
Install Test English (GB) Sample Data demo.
Set Search Engine Friendly URLs to No.
On the front end, click Archived Articles.
In the browser's address bar, append &catid=0 to the URL.
See warnings above the filter form.
Expected result
No warnings.
Actual result
Warning: array_diff(): Argument #1 is not an array in C:\xampp\htdocs\joomla-cms-staging\components\com_content\models\archive.php on line 90
Warning: array_values() expects parameter 1 to be array, null given in C:\xampp\htdocs\joomla-cms-staging\components\com_content\models\archive.php on line 90
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19075.
| Status | Pending | ⇒ | Ready to Commit |
Ready to Commit after two successful tests.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-12-18 03:30:48 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
?
|
||
I have tested this item✅ successfully on 0ccf28b
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19075.