Feeling Lucky
?
[#7980] - Redo of PR 6323
- Closed
- 8 May 2016
- Medium
- Build: staging
- # 7980
- Diff
- Mathewlenning:CLEANING-JFormField-Redo
- Failure Human Test Results: 0 Successful 2 Failed. Details
Pull Request for
# 6323
User tests: Successful: Unsuccessful:
Simple cleaning of a few redundant conditionals
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
zero-24
- | Rel_Number | 0 | ⇒ | 6323 |
| Relation Type | ⇒ | Pull Request for |
| Category | ⇒ | Libraries |
pos-it
- comment
- 24 Oct 2015
I have tested this item 
unsuccessfully on aa2bd87
I used the Module mod_qlform v9.3.1 that create cusomed Forms.
As soon as I activate this patch a get a Fatal Error.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7980.
si-wi
- comment
- 24 Oct 2015
I have tested this item unsuccessfully on aa2bd87
the new version of getAttribute will never return an attributes value, but always default (null)
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7980.
Kubik-Rubik
- comment
- 8 May 2016
Thank you for creating this. It’s been some time since you created this and there are now some merge conflicts that prevent a direct merge and also 2 unsuccessful tests. I will close this PR now. If you want to update your PR, then this can be re-opened.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7980.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-05-08 10:34:16 |
| Closed_By | ⇒ | Kubik-Rubik |
Please add a testing instruction
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7980.