Feeling Lucky
?
?
?
[#14247] - Skip delete when id doesn’t exists
- Fixed in Code Base
- 3 Mar 2017
- Medium
- Build: staging
- # 14247
- Diff
- zero-24:zero-24-patch-3
User tests: Successful: Unsuccessful:
Pull Request for Issue c0eadd9
Summary of Changes
Just spaces to tabs + two additional inline comments
Testing Instructions
- Just review
- or install the patch and run the database fixer
. confirm it still works.
Expected result
match the code standards
Actual result
does not match the code standards
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_admin |
| Labels |
Added:
?
?
|
||
richard67
- comment
- 25 Feb 2017
P.S.: And of course I can also confrirm it matches code standards now.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14247.
Sieger66
- comment
- 3 Mar 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14247.
franz-wohlkoenig
- comment
- 3 Mar 2017
RTC as there are 2 successfully Tests?
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-03 14:55:03 |
| Closed_By | ⇒ | rdeutz | |
| Labels | |||
I have tested this item✅ successfully on 270e8ca
Tested by code review. I can confirm it only changes code style and adds useful comments.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14247.