?
Success
-
Success
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
-
Success
JTracker/HumanTestResults
Human Test Results: 2 Successful 0 Failed.
Details
User tests:
Successful:
Unsuccessful:
Code standards (https://joomla.github.io/coding-standards/?coding-standards/chapters/xml.md) + correction closing tags, form xml files com_contact
Summary of Changes
config.xml
- Code standards
- Add emtpy line before each opening tag (readability)
- Fix wrong field closing tag :
image
models/forms/contact.xml
- Code standards
- Add emtpy line before each opening tag (readability)
- Fix wrong field closing tag :
tags
- Fix mix space/tabs for a few indents
models/forms/filter_contacts.xml
- Code standards
- Add emtpy line before each opening tag (readability)
Testing Instructions
- code review
- apply and test all is working as before
JoomliC
-
open
-
15 May 2016
JoomliC
-
change
-
15 May 2016
joomla-cms-bot
-
change
-
15 May 2016
brianteeman
-
change
-
16 May 2016
andrepereiradasilva
-
test_item
-
16 May 2016
-
Tested successfully
freshweb
-
test_item
-
21 May 2016
-
Tested successfully
brianteeman
-
change
-
21 May 2016
Status |
Pending |
⇒ |
Ready to Commit |
joomla-cms-bot
-
change
-
21 May 2016
brianteeman
-
change
-
21 May 2016
roland-d
-
change
-
21 May 2016
Status |
Ready to Commit |
⇒ |
Fixed in Code Base |
Closed_Date |
0000-00-00 00:00:00 |
⇒ |
2016-05-21 16:35:48 |
Closed_By |
|
⇒ |
roland-d |
roland-d
-
close
-
21 May 2016
roland-d
-
merge
-
21 May 2016
joomla-cms-bot
-
close
-
21 May 2016
roland-d
-
merge
-
21 May 2016
roland-d
-
close
-
21 May 2016
joomla-cms-bot
-
change
-
21 May 2016
Add a Comment
Login with GitHub to post a comment
I have tested this item successfully on 5e84871
on code review
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10504.