Success

User tests: Successful: Unsuccessful:

avatar Buddhima
Buddhima
23 Sep 2013

Hi All,
This PR consists,

  1. com_config in new MVC
  2. front-end com_config, which consumes services from back end com_config and com_templates
  3. library classes related to new MVC ** Implemented within the component **

Feature Tracker: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_id=8549&tracker_item_id=31264

Note: This is a reorganized version of com_services project.Previous one has files in library/cms , but in this new one library files are inside component folder.

Thank You !

avatar Buddhima Buddhima - open - 23 Sep 2013
avatar wilsonge wilsonge - reference | - 28 Sep 13
avatar brianteeman
brianteeman - comment - 30 Sep 2013

If you dont set the correct user level then the menutype "Configuration Manager » config" will give two error messages "You are not authorised to view this resource" BUT the menutyoe "Configuration Manager » templates" correctly only gives one

avatar Buddhima
Buddhima - comment - 1 Oct 2013

Hi Brian Teeman,

I have fixed the issue. This was caused becaused of enququing error messages twice by frontend and backend display controllers.

Thank You !

avatar brianteeman
brianteeman - comment - 1 Oct 2013

I'm struggling to work out the logic for which fields you have chosen to display for site configuration. For templates you appear to have chosen everything under the options tab but for site it appears to be more arbitrary. Can you explain the reasoning please.

avatar Buddhima
Buddhima - comment - 1 Oct 2013

Hi Brian Teeman,
Selection of the fields in Config is done by the backend json view file.
Link: https://github.com/Buddhima/joomla-cms/blob/com_config_gsoc/administrator/components/com_config/view/application/json.php#L54
We've selected to display only those fields after some discussions too.
Link: https://docs.google.com/spreadsheet/ccc?key=0Al8pu0AnookwdGh3MGNtVElXYTdfS0JUbTBTNEZkVEE&usp=sharing

About template, first it look for com_config.xml file frst:
https://github.com/Buddhima/joomla-cms/blob/com_config_gsoc/components/com_config/model/templates.php#L107
If not present search for config in templateDetails.xml of the template.
This is done because, some templates failed to show some fields, so that developers have chance to specify which fields to display through com_config.xml file.

Hope I gave a clear explanation regarding the question you have and feel free to ask if there's any concern regarding this.

Thank You !

avatar brianteeman
brianteeman - comment - 1 Oct 2013

Thanks for the explanation.
Site
With the limited set of options I just don't see why anyone would ever use it or need to use it

template
With the current implementation it is not usable on a site with multiple templates or multiple template styles as the menu item only loads the options for the template style that is active for that menu item

avatar mbabker
mbabker - comment - 9 Oct 2013

Can you sync up to master again please?

avatar Buddhima
Buddhima - comment - 9 Oct 2013

Hi Micheal, Not this is synced again. Thank You!

avatar mbabker mbabker - close - 12 Oct 2013
avatar Buddhima
Buddhima - comment - 13 Oct 2013

Hi All,

Thank You Micheal for merging this.

Thank You Elin, Ofer, Chad, David, Nick, Michael, George and Brian for spending time on inspecting, testing and guiding me to produce good quality feature.

I believe new MVC and front-end com_config will help to make Joomla! CMS better.

Thank You !

avatar garyamort garyamort - reference | - 2 Dec 13

Add a Comment

Login with GitHub to post a comment