? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
8 Nov 2020

Fixes obvious typing/spelling error BASIS should be BASIC

code review

This is a 3.x version of #31361 - it has to be done as separate PR because just merging this one into 4 will not fix everything in 4

avatar brianteeman brianteeman - open - 8 Nov 2020
avatar brianteeman brianteeman - change - 8 Nov 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Nov 2020
Category Administration Language & Strings Front End com_users
avatar richard67 richard67 - test_item - 8 Nov 2020 - Tested successfully
avatar richard67
richard67 - comment - 8 Nov 2020

I have tested this item successfully on 68edb4a


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

avatar richard67
richard67 - comment - 8 Nov 2020

@ceford As you have tested the other one for J4, can you test this one here for J3, too? Thanks in advance.

avatar SharkyKZ
SharkyKZ - comment - 8 Nov 2020

Isn't removing strings a B/C break?

avatar richard67
richard67 - comment - 8 Nov 2020

Oh yes.

avatar richard67 richard67 - test_item - 8 Nov 2020 - Not tested
avatar richard67
richard67 - comment - 8 Nov 2020

I have not tested this item.


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

avatar brianteeman
brianteeman - comment - 8 Nov 2020

Oops I forgot

Closing

avatar brianteeman brianteeman - close - 8 Nov 2020
avatar brianteeman brianteeman - change - 8 Nov 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-11-08 10:34:55
Closed_By brianteeman
Labels Added: ? ?
avatar richard67
richard67 - comment - 8 Nov 2020

Would it make sense to do this fix for J3, only not deleting the old language strings but marking with a deprecation comment line above them, telling to use the new ones, and adding the new language strings (new constants)?

avatar brianteeman
brianteeman - comment - 8 Nov 2020

Dont think its worth it

Add a Comment

Login with GitHub to post a comment