Language Change PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar MacJoom
MacJoom
28 Mar 2024

Postinstallation-Message for Brotli compression bug #43055

Pull Request for Issue # .

Summary of Changes

Testing Instructions

Actual result BEFORE applying this Pull Request

avatar MacJoom MacJoom - open - 28 Mar 2024
avatar MacJoom MacJoom - change - 28 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2024
Category Administration com_admin SQL Postgresql Language & Strings
avatar richard67 richard67 - change - 28 Mar 2024
Labels Added: Language Change PR-4.4-dev
avatar richard67 richard67 - test_item - 28 Mar 2024 - Tested successfully
avatar richard67
richard67 - comment - 28 Mar 2024

I have tested this item ✅ successfully on e3200de

I've tested updating to the patched package on both MySQL and PostgreSQL. In both cases it worked, and the postinstall message was added.


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

avatar brianteeman
brianteeman - comment - 28 Mar 2024

not sure what will happen when someone who has updated to 4.4.4 then goes to 5.1 - worse case i think is that they will see the message twice

avatar richard67
richard67 - comment - 28 Mar 2024

not sure what will happen when someone who has updated to 4.4.4 then goes to 5.1 - worse case i think is that they will see the message twice

@brianteeman Check my comments in your PR. The message will not be duplicated as we have the INSERT IGNORE/ON CONFLICT DO NOTHING.

avatar brianteeman
brianteeman - comment - 28 Mar 2024

which is why I said "not sure" ;)

avatar richard67 richard67 - alter_testresult - 28 Mar 2024 - richard67: Tested successfully
avatar laoneo laoneo - close - 28 Mar 2024
avatar laoneo laoneo - merge - 28 Mar 2024
avatar laoneo laoneo - change - 28 Mar 2024
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-03-28 15:56:51
Closed_By laoneo
avatar laoneo
laoneo - comment - 28 Mar 2024

Thanks!

Add a Comment

Login with GitHub to post a comment