Feeling Lucky
?
[#9403] - Changing line 230 on cms.php
- Fixed in Code Base
- 13 Apr 2016
- Medium
- Build: staging
- # 9403
- Diff
- sapphireweb:patch-1
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
Pull Request for
# 9381
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Changing line 230 of cms.php to handle non-empty, blank messages.
Testing Instructions
The current code, if (!strlen($msg)) still allows blank messages to be added to the messageQue. Changing line 230 to, if (!strlen(trim($msg))) removes whitespace before checking the length, insuring that a message full just spaces won't be added,
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
zero-24
- comment
- 13 Mar 2016
I have tested this item 
successfully on a12f627
Works here see my test instructions. Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9403.
Twincarb
- comment
- 13 Mar 2016
I have tested this item successfully on a12f627
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9403.
| Rel_Number | 0 | ⇒ | 9381 |
| Relation Type | ⇒ | Pull Request for |
| Status | Pending | ⇒ | Ready to Commit |
brianteeman
- comment
- 13 Mar 2016
| Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
| Labels | |||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-04-13 05:23:10 |
| Closed_By | ⇒ | rdeutz | |
| Labels | |||
| Labels |
Removed:
?
|
||
How to test
JFactory::getApplication()->enqueueMessage(' ', 'error');