? Success
Pull Request for # 9521

User tests: Successful: Unsuccessful:

avatar Twincarb
Twincarb
23 Mar 2016

Pull Request for Issue #9521 .

Summary of Changes

set default install value for mod_breadcrumbs caching value to off '0' also caching time to '0' so that it has to be a definite wish to enable it.

Testing Instructions

This PR will only have an effect from a clean install on selecting a new breadcrumb menu, going to the advanced settings cache should by default be disabled and caching time 0

I need to find the sample install data and check to see if it's enabled at all please hold off testing at the moment, but I welcome comments.

avatar Twincarb Twincarb - open - 23 Mar 2016
avatar Twincarb Twincarb - change - 23 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Mar 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 23 Mar 2016
Rel_Number 0 9521
Relation Type Pull Request for
avatar brianteeman brianteeman - change - 23 Mar 2016
Category Cache Modules
avatar infograf768
infograf768 - comment - 23 Mar 2016

Yep, it also needs chnages in the joomla.sql for each db:

(17, 51, 'Breadcrumbs', '', '', 1, 'position-2', 0, '0000-00-00 00:00:00', '0000-00-00 00:00:00', '0000-00-00 00:00:00', 1, 'mod_breadcrumbs', 1, 1, '{"moduleclass_sfx":"","showHome":"1","homeText":"","showComponent":"1","separator":"","cache":"1","cache_time":"900","cachemode":"itemid"}', 0, '*'),
avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Mar 2016

Yes in all db drivers
And it should be changed in sample data database SQL too.

avatar Twincarb
Twincarb - comment - 23 Mar 2016

Daft question, how do I update my PR?

avatar Twincarb Twincarb - change - 23 Mar 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-03-23 13:11:52
Closed_By Twincarb
avatar Twincarb Twincarb - close - 23 Mar 2016
avatar Twincarb Twincarb - close - 23 Mar 2016
avatar Twincarb Twincarb - change - 23 Mar 2016
Status Closed New
Closed_Date 2016-03-23 13:11:52
Closed_By Twincarb
avatar Twincarb Twincarb - change - 23 Mar 2016
Status New Pending
avatar Twincarb Twincarb - reopen - 23 Mar 2016
avatar Twincarb Twincarb - reopen - 23 Mar 2016
avatar brianteeman
brianteeman - comment - 23 Mar 2016

Just update your branch and the pull request will be updated automatically


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Mar 2016

you have to update the files in your "patch-2" branch
https://github.com/Twincarb/joomla-cms/tree/patch-2

any changes you made in that branch will automatically will be in this PR.

avatar Twincarb
Twincarb - comment - 23 Mar 2016

I think that's all the files that needed adjusting, I can't find the location for adjusting the tool tip for caching on the advanced tab, it anyone can point me in the right direction...


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Mar 2016

that string (COM_MODULES_FIELD_CACHING_DESC) is global for all modules.

the location is https://github.com/joomla/joomla-cms/blob/staging/administrator/language/en-GB/en-GB.com_modules.ini#L37

avatar Twincarb
Twincarb - comment - 23 Mar 2016

@andrepereiradasilva Thanks for the pointer, I was thinking it was possibly going to be a global string, it's beyond my ability to set it up to something separate for the one module.
The db drivers, which files am I looking for, all of the sql folder files are updated.


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

avatar andrepereiradasilva andrepereiradasilva - test_item - 23 Mar 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Mar 2016

I have tested this item :white_check_mark: successfully on 760bfcb

Tested by code review


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

avatar brianteeman brianteeman - test_item - 24 Mar 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 24 Mar 2016

I have tested this item :white_check_mark: successfully on 760bfcb


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

avatar brianteeman brianteeman - change - 25 Mar 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 25 Mar 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 25 Mar 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 12 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-12 22:52:11
Closed_By wilsonge
avatar wilsonge wilsonge - close - 12 Apr 2016
avatar wilsonge wilsonge - merge - 12 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 12 Apr 2016
avatar wilsonge wilsonge - reference | 19270af - 12 Apr 16
avatar wilsonge wilsonge - merge - 12 Apr 2016
avatar wilsonge wilsonge - close - 12 Apr 2016
avatar wilsonge wilsonge - change - 12 Apr 2016
Milestone Added:
avatar joomla-cms-bot joomla-cms-bot - change - 12 Apr 2016
Labels Removed: ?
avatar wilsonge
wilsonge - comment - 12 Apr 2016

Merged - thanks!

avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment