? ? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
23 Aug 2015

This fixes: #7751 Thanks @tranduyhung and @infograf768


There are 2 language keys which I can't find them anywhere in Joomla language files

JLIB_APPLICATION_ERROR_CREATE_NOT_PERMITTED - used in

https://github.com/joomla/joomla-cms/blob/master/administrator/components/com_media/controllers/folder.php#L168

JERROR_CORE_CREATE_NOT_PERMITTED - used in

https://github.com/joomla/joomla-cms/blob/master/administrator/components/com_templates/models/style.php#L169 and https://github.com/joomla/joomla-cms/blob/master/administrator/components/com_modules/models/module.php#L398

If they are really missing. What language ini file should we add them to? Can we just simply say "Create not permitted." just like in com_media?

COM_MEDIA_ERROR_CREATE_NOT_PERMITTED="Create not permitted."
avatar zero-24 zero-24 - open - 23 Aug 2015
avatar zero-24 zero-24 - change - 23 Aug 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2015
Labels Added: ? ?
avatar chrisdavenport
chrisdavenport - comment - 23 Aug 2015

Missing full stops at the end? Do we have a standard for that?

avatar chrisdavenport
chrisdavenport - comment - 23 Aug 2015

Thanks. :-)

avatar zero-24 zero-24 - change - 23 Aug 2015
Category Language & Strings
avatar zero-24 zero-24 - change - 23 Aug 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 23 Aug 2015

Based on @chrisdavenport 's en-GB Review and the comments by @tranduyhung and @infograf768 on #7751 I'm going to RTC here. Thanks!


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

avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2015
Labels Added: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 24 Aug 2015

@zero-24 You use two different language constants. Is it on purpose?

avatar Kubik-Rubik Kubik-Rubik - change - 24 Aug 2015
Status Ready to Commit Information Required
avatar Kubik-Rubik Kubik-Rubik - change - 24 Aug 2015
Labels Removed: ?
avatar infograf768
infograf768 - comment - 24 Aug 2015

Yes. One exists already in lang packs and can't be used globally. The new one is global.

avatar Kubik-Rubik
Kubik-Rubik - comment - 24 Aug 2015

@infograf768 Okay, I understand. Then it's good to be merged.

Thank you @zero-24!

avatar Kubik-Rubik Kubik-Rubik - change - 24 Aug 2015
Status Information Required Ready to Commit
avatar Kubik-Rubik Kubik-Rubik - change - 24 Aug 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-08-24 08:13:09
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - reference | bff21ab - 24 Aug 15
avatar Kubik-Rubik Kubik-Rubik - merge - 24 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - close - 24 Aug 2015
avatar Kubik-Rubik Kubik-Rubik - change - 24 Aug 2015
Milestone Added:
avatar zero-24 zero-24 - head_ref_deleted - 24 Aug 2015

Add a Comment

Login with GitHub to post a comment