? ? Failure

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
23 Jul 2016

This mainly removes use of the word "whether" and standardises some strings to use "show or hide". It is simpler English and more consistent across all the strings

Note if you grep the language files for the word "whether" there may be some instances still present because those language strings are not used any more in core.

Testing - please check for typos

avatar brianteeman brianteeman - open - 23 Jul 2016
avatar brianteeman brianteeman - change - 23 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jul 2016
Category Language & Strings Administration
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jul 2016
Labels Added: ? ?
c03e7a8 23 Jul 2016 avatar brianteeman oops
f460de7 23 Jul 2016 avatar brianteeman comma
c17f571 23 Jul 2016 avatar brianteeman caps
avatar zero-24 zero-24 - test_item - 23 Jul 2016 - Tested successfully
avatar zero-24
zero-24 - comment - 23 Jul 2016

I have tested this item successfully on 19e40ea

looks good now


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

avatar jeckodevelopment
jeckodevelopment - comment - 24 Jul 2016

I have tested this item successfully on 19e40ea

avatar brianteeman brianteeman - change - 24 Jul 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 24 Jul 2016

RTC - thanks for testing


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

avatar joomla-cms-bot joomla-cms-bot - change - 24 Jul 2016
Labels Added: ?
avatar rdeutz
rdeutz - comment - 13 Aug 2016

@brianteeman could you have a look at the merge conflicts, thanks.

avatar brianteeman
brianteeman - comment - 13 Aug 2016

@rdeutz yeah I was waiting for all the other PR to be merged first as I would have had to do it again

avatar brianteeman
brianteeman - comment - 13 Aug 2016

I give up - my local git say it is upto date

avatar brianteeman
brianteeman - comment - 14 Aug 2016

@zero-24 can you take a look at this one as well please - one day I will understand git but that day is not today

avatar brianteeman
brianteeman - comment - 14 Aug 2016

@zero-24 thanks for fixing the merge conflicts

@rdeutz this can be merged now - thanks

avatar truptikagathara truptikagathara - test_item - 15 Aug 2016 - Tested successfully
avatar truptikagathara
truptikagathara - comment - 15 Aug 2016

I have tested this item successfully on 62b7b90


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

avatar rdeutz
rdeutz - comment - 16 Aug 2016

Travis fails because a script has moved, I going to merge this anyway, hoping the internet will not implode

avatar rdeutz rdeutz - change - 16 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-16 21:06:08
Closed_By rdeutz
avatar brianteeman
brianteeman - comment - 17 Aug 2016

Thanks

avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2016
Labels Removed: ?
avatar infograf768
infograf768 - comment - 17 Aug 2016

+MOD_LANGUAGES_FIELD_CACHING_DESC="Use the global cache setting the content of this module or disable caching for this module.<br />This should be set to 'No caching' when using Items Associations."
"setting the content" does not make sense to me.

Should'nt it be?
+MOD_LANGUAGES_FIELD_CACHING_DESC="Use the global cache setting to cache the content of this module or disable caching for this module.<br />This should be set to 'No caching' when using Items Associations."

avatar mbabker
mbabker - comment - 17 Aug 2016

Somewhat off topic but why can't this module be cached with associations in use? That seems like a bug.

avatar jeckodevelopment
jeckodevelopment - comment - 17 Aug 2016

@infograf768 I opened PR #11641 to fix the string.
@mbabker I'm not sure I got your report

avatar mbabker
mbabker - comment - 17 Aug 2016

The language string JM's pointing out says that mod_languages should have caching disabled if item associations are in use. That doesn't seem right at all.

Add a Comment

Login with GitHub to post a comment