? Success

User tests: Successful: Unsuccessful:

avatar JoomliC
JoomliC
15 May 2016

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
avatar JoomliC JoomliC - open - 15 May 2016
avatar JoomliC JoomliC - change - 15 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 16 May 2016
Category Code style
avatar andrepereiradasilva andrepereiradasilva - test_item - 16 May 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 16 May 2016

I have tested this item :white_check_mark: successfully on 5e84871

on code review


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10504.

avatar freshweb freshweb - test_item - 21 May 2016 - Tested successfully
avatar freshweb
freshweb - comment - 21 May 2016

I have tested this item successfully on 5e84871


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10504.

avatar brianteeman brianteeman - change - 21 May 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 21 May 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10504.

avatar joomla-cms-bot joomla-cms-bot - change - 21 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 21 May 2016
Milestone Added:
avatar roland-d 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
avatar roland-d roland-d - close - 21 May 2016
avatar roland-d roland-d - merge - 21 May 2016
avatar joomla-cms-bot joomla-cms-bot - close - 21 May 2016
avatar roland-d roland-d - reference | f2e9928 - 21 May 16
avatar roland-d roland-d - merge - 21 May 2016
avatar roland-d roland-d - close - 21 May 2016
avatar joomla-cms-bot joomla-cms-bot - change - 21 May 2016
Labels Removed: ?
avatar roland-d
roland-d - comment - 21 May 2016

Thanks everybody


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10504.

Add a Comment

Login with GitHub to post a comment