-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[4.0]Namespace mod multilangstatus #21020
Conversation
@infograf768 Please test this PR for your issue with joomla#21009
Bravo. That was it.
|
@@ -9,6 +9,7 @@ | |||
<authorUrl>www.joomla.org</authorUrl> | |||
<version>3.0.0</version> | |||
<description>MOD_MULTILANGSTATUS_XML_DESCRIPTION</description> | |||
<namespace>Joomla\Module\Multilangstatus</namespace> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not indented for other core modules
I was waiting to see if it worked for you. Will update soon |
Note: I am curious to learn where in code this new setting in the xml is used when installing a clean Joomla. |
I have tested this item ✅ successfully on 5e03cc8 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/21020. |
OOps. No. the update sql are not good but |
sorry - copy paste error and then forgetting to save the change |
OK now. One more tester. |
@@ -0,0 +1 @@ | |||
UPDATE `#__extensions` SET `namespace` = 'Joomla\\Module\\Multilangstatus' WHERE `name` = 'mod_multilangstatus'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need the update scripts as this is a new feature in J4.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i thought that too but look at the other update scripts - :)
maybe we should just make a tracker item to review all update scripts before release
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@laoneo
What would happen if the db is not updated when updating Joomla?
Also, please explain about:
I am curious to learn where in code this new setting in the xml is used when installing a clean Joomla.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My bad, we need the updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we should just make a tracker item to review all update scripts before release
#16458 is already there. Though it's far enough down the list I fully expect duplicates to be opened down the line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just added a release blocker tag on it so it will be easier to find. I hope.
Thanks |
@infograf768 Please test this PR for your issue with #21009