? ? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
1 May 2017

When installing a language where the respective content language already exists (eg due to uninstalling the language previously), you will currently get a warning that the content language couldn't be created automatically because there is already one with that tag.
A warning seems to be a bit harsh for something that is actually working just fine. A notice is more than enough.

Summary of Changes

Changing the type of the message to a notice.

Testing Instructions

  • Install a language, uninstall it and install it again

Expected result

Get a notice that the content language already exists.

Actual result

Get a warning that the content language already exists.

Documentation Changes Required

None

avatar Bakual Bakual - open - 1 May 2017
avatar Bakual Bakual - change - 1 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 May 2017
Category Libraries
avatar syntaxerror
syntaxerror - comment - 1 May 2017

Tested the code change successfully on 3.7.0

avatar franz-wohlkoenig franz-wohlkoenig - alter_testresult - 2 May 2017 - syntaxerror: Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 May 2017

@syntaxerror i have set your Test as successfully in Issue Tracker.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 May 2017

I have tested this item successfully on 1d44524


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 2 May 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 2 May 2017
Status Pending Ready to Commit
Easy No Yes
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 May 2017

RTC after two successful tests.

avatar wilsonge wilsonge - change - 3 May 2017
Milestone Added:
avatar rdeutz rdeutz - close - 22 May 2017
avatar rdeutz rdeutz - merge - 22 May 2017
avatar rdeutz rdeutz - change - 22 May 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-22 18:44:50
Closed_By rdeutz
Labels Added: ? ?

Add a Comment

Login with GitHub to post a comment