[#9285] - set() returns the old value of the variable #9276 (revised)
- Closed
- 2 Mar 2016
- Medium
- Build: staging
- # 9285
- Diff
- GABBAR1947:t1
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #8859
Testing instructions remain the same as referenced in the issue:
$session = JFactory::getSession();
$session->set('test', 1);
echo $session->set('test', 2);
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
I have tested this item 
successfully on 21e9fec
Works as described.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
@richard67 This is the first time I'm facing such kind of issues on github. It's absolutely fine on my system, must be a github issue, as I indented on github itself in the second commit.
This PR has received new commits.
CC: @richard67
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
I have tested this item successfully on 21e9fec
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
This PR has received new commits.
CC: @richard67
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
I have tested this item successfully on 21e9fec
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
| Labels |
Added:
?
|
||
| Labels |
Added:
?
|
||
| Labels |
Removed:
?
|
||
I have tested this item successfully on 21e9fec
Test OK
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.
Merged
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-03-02 22:04:33 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
@GABBAR1947 Seems you have indented with 8 spaces and the 1 tab instead of 2 tabs? Or was it edited on GitHub, and GitHub did that?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9285.