User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Fixed the default value for position and currency.
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.
"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.
No "2" visible when currency is empty. And default position is still in front of the text (0).
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
| Status | New | ⇒ | Pending |
| Category | ⇒ | Front End Plugins |
@brianteeman maybe i don't understand your comment but afaik i did add them.
See here:
joomla-cms/installation/sql/mysql/base.sql
Line 302 in 5e7bf48
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.
I have tested this item ✅ successfully on 7a1db96
| Labels |
Added:
PR-6.0-dev
|
||
| Category | Front End Plugins | ⇒ | SQL Installation Front End Plugins |
| Category | Front End Plugins SQL Installation | ⇒ | SQL Installation Postgresql Front End Plugins |
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.
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.
Thanks for the explanation, that's fine for me. 👍🏼 An override can be done at any time if someone wants a space between.
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.
But the options are missing the "symbol" value.
Thats what I meant
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.
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.
I have tested this item ✅ successfully on cb652ce
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.
I have tested this item ✅ successfully on cb652ce
Tested Successfully. 9319e04ad Fix works as expected.
| Status | Pending | ⇒ | Ready to Commit |
RTC
| Labels |
Added:
RTC
|
||
| 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 |
Thanks for the fix @TLWebdesign and to @LadySolveig & @lemuelvdm for testing at short notice!
Why were the defautl values not set in the database as they are with the other params