RTC PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar TLWebdesign
TLWebdesign
10 Nov 2025

Pull Request for Issue # .

Summary of Changes

Fixed the default value for position and currency.

Testing Instructions

When no currency is set it should not display anything. Now it display a "2".
When no position is set it should default back to 0 (in front) which is the same as the default in the XML.

Actual result BEFORE applying this Pull Request

"2" was displayed when no currency was set.
Default position was set to "2" which would give the same behaviour as setting it to the back, altho this was technically not possible since the default position in the XML was already set to 0 and it is a list so no user input possible.

Expected result AFTER applying this Pull Request

No "2" visible when currency is empty. And default position is still in front of the text (0).

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar TLWebdesign TLWebdesign - open - 10 Nov 2025
avatar TLWebdesign TLWebdesign - change - 10 Nov 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Nov 2025
Category Front End Plugins
avatar brianteeman
brianteeman - comment - 10 Nov 2025

Why were the defautl values not set in the database as they are with the other params

avatar TLWebdesign
TLWebdesign - comment - 10 Nov 2025

@brianteeman maybe i don't understand your comment but afaik i did add them.

See here:

(0, 'plg_fields_number', 'plugin', 'number', 'fields', 0, 1, 1, 0, 1, '', '{"min":"1.0","max":"100.0","step":"0.1","currency":"0","position":"0","decimals":"2"}', '', 10, 0),

avatar richard67
richard67 - comment - 10 Nov 2025

Why were the defautl values not set in the database as they are with the other params

@brianteeman In fact they were added to the SQL with PR #43974 .

But the options are missing the "symbol" value.

@TLWebdesign Not sure if we need an update SQL to fix that in database. but we should at least fix base.sql so it is right from the beginning on for new installations.
Update: I think we will not need a new update SQL for fixing it in the database on updated sites. For those the missing value in the options in database will cause the default value being used in PHP. But in base.sql it should be right.

avatar LadySolveig LadySolveig - test_item - 10 Nov 2025 - Tested successfully
avatar LadySolveig
LadySolveig - comment - 10 Nov 2025

I have tested this item ✅ successfully on 7a1db96


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

avatar TLWebdesign TLWebdesign - change - 10 Nov 2025
Labels Added: PR-6.0-dev
avatar joomla-cms-bot joomla-cms-bot - change - 10 Nov 2025
Category Front End Plugins SQL Installation Front End Plugins
avatar joomla-cms-bot joomla-cms-bot - change - 10 Nov 2025
Category Front End Plugins SQL Installation SQL Installation Postgresql Front End Plugins
avatar LadySolveig
LadySolveig - comment - 10 Nov 2025

It still needs a little improvement. Currently, there is no space between the number and the currency symbol. And yes, the database value for the currency symbol is missing.
The fix resolves the error for the user. If you can submit the fix for the DB at short notice, I'll be happy to renew my test again.

avatar TLWebdesign
TLWebdesign - comment - 10 Nov 2025

Hi all the SQL is changed.

@LadySolveig not adding a space is intentional as some currencies don't use spaces in between so now you get to choose to add a space or not.

avatar LadySolveig
LadySolveig - comment - 10 Nov 2025

Thanks for the explanation, that's fine for me. 👍🏼 An override can be done at any time if someone wants a space between.

avatar TLWebdesign
TLWebdesign - comment - 10 Nov 2025

they don't need to do an override. they can just type a space after the symbol and then there is a space in between the symbol and the number.

avatar brianteeman
brianteeman - comment - 10 Nov 2025

But the options are missing the "symbol" value.

Thats what I meant

avatar TLWebdesign
TLWebdesign - comment - 10 Nov 2025

But the options are missing the "symbol" value.

Thats what I meant

Yes thank you i overlooked that until richard pointed it out. i fixed it now :D thanks for noticing. Still much to learn.

avatar richard67
richard67 - comment - 10 Nov 2025

But the options are missing the "symbol" value.

Thats what I meant

@brianteeman For new installations it has been fixed meanwhile in the base.sql. I think it does not need a new update SQL for updated sites. FOr these we still have the right PHP default.

avatar LadySolveig LadySolveig - test_item - 10 Nov 2025 - Tested successfully
avatar LadySolveig
LadySolveig - comment - 10 Nov 2025

I have tested this item ✅ successfully on cb652ce


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

avatar richard67
richard67 - comment - 10 Nov 2025

Tested 9319e04 successfully.

@lemuelvdm Please mark your test result in the issue tracker here https://issues.joomla.org/tracker/joomla-cms/46433 by using the blue "Test this" button at the top left corner, selecting your test result and finally submit, so the test is properly counted. Thanks in advance.

avatar lemuelvdm lemuelvdm - test_item - 10 Nov 2025 - Tested successfully
avatar lemuelvdm
lemuelvdm - comment - 10 Nov 2025

I have tested this item ✅ successfully on cb652ce

Tested Successfully. 9319e04ad Fix works as expected.


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

avatar richard67 richard67 - change - 10 Nov 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 10 Nov 2025

RTC


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

avatar Bodge-IT Bodge-IT - change - 10 Nov 2025
Labels Added: RTC
avatar Bodge-IT Bodge-IT - change - 10 Nov 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-11-10 23:38:04
Closed_By Bodge-IT
avatar Bodge-IT Bodge-IT - close - 10 Nov 2025
avatar Bodge-IT Bodge-IT - merge - 10 Nov 2025
avatar Bodge-IT
Bodge-IT - comment - 10 Nov 2025

Thanks for the fix @TLWebdesign and to @LadySolveig & @lemuelvdm for testing at short notice!

Add a Comment

Login with GitHub to post a comment