Skip to content

Fix user params after Joomla 3.9.7 with selected french help site#25177

Merged
HLeithner merged 3 commits intojoomla:stagingfrom
HLeithner:fix-french-helpsite-removal
Jun 11, 2019
Merged

Fix user params after Joomla 3.9.7 with selected french help site#25177
HLeithner merged 3 commits intojoomla:stagingfrom
HLeithner:fix-french-helpsite-removal

Conversation

@HLeithner
Copy link
Copy Markdown
Member

Pull Request for Issue #25176

Summary of Changes

Tries to fix user table if json got corrupted.

@alikon

@richard67
Copy link
Copy Markdown
Member

The old schema updates 3.9.7-2019-05-16.sql should be corrected, too, so we in future don't create the error which new schema updates provided by this PR shall fix on other sites. Problem is that I don't know if the helpsite parameter always is placed somewhere in the middle of the json, or if it also can be first or last parameter in the list.

@mbabker
Copy link
Copy Markdown
Contributor

mbabker commented Jun 11, 2019

I still say the param shouldn't be touched. Empty the 3.9.7-2019-05-16 changeset and have this change to fix the broken database fields. It was getting rid of the param that broke things in the first place and it was an unnecessary action.

@richard67
Copy link
Copy Markdown
Member

@mbabker

... have this change to fix the broken database fields.

You mean like in this PR, with new schema update files? Or just as howto in a forum?

Is the change in this PR here safe? Or can it be that the helpsite parameter is not in the middle but at the beginning or the end of the json list so we need a more sophisticated replacement?

@HLeithner
Copy link
Copy Markdown
Member Author

I removed the content from the old sql file as michael suggested.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jun 11, 2019

yes make sense to remove the buggy replace imo

@mbabker
Copy link
Copy Markdown
Contributor

mbabker commented Jun 11, 2019

Should be good now.

@richard67
Copy link
Copy Markdown
Member

If helpsite can also be at beginning or end of the json, we also have to replace {, with {and ,} with }.

@SniperSister
Copy link
Copy Markdown
Contributor

I tested the patch on an updated 3.9.7, works fine and restores the JSON

@HLeithner
Copy link
Copy Markdown
Member Author

normally the helpsite is between 2 other parameters I don't think that it's possible that the ,, are at the beginning or the end...

@richard67
Copy link
Copy Markdown
Member

Should the obsolete schema updates which are deleted with this PR also be deleted on already updated sites, i.e. added somehwre here https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_admin/script.php#L1989 ?

@HLeithner
Copy link
Copy Markdown
Member Author

It will be overridden anyway, not sure if it makes sense to remove it.

@mbabker
Copy link
Copy Markdown
Contributor

mbabker commented Jun 11, 2019

It's fine as an empty file. Or change the file contents to have a SQL comment that way it's clear why there are no queries in it.

# Query removed, see https://github.com/joomla/joomla-cms/pull/25177

@richard67
Copy link
Copy Markdown
Member

I like Michael's suggestion in his previous comment, have a comment in the wrong sql file.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jun 11, 2019

lgtm

@HLeithner
Copy link
Copy Markdown
Member Author

I added the comment.

@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 872c9ce

Tested with 3.9.7 site with user having parameters with error caused by wrong helspite removal.


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

Copy link
Copy Markdown
Member

@richard67 richard67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Jun 11, 2019

I have tested this item ✅ successfully on 872c9ce


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

@HLeithner HLeithner merged commit 941631d into joomla:staging Jun 11, 2019
@HLeithner HLeithner added this to the Joomla 3.9.8 milestone Jun 11, 2019
@HLeithner
Copy link
Copy Markdown
Member Author

thx for the help, release is out.

@HLeithner HLeithner deleted the fix-french-helpsite-removal branch June 11, 2019 20:46
@pescweb
Copy link
Copy Markdown

pescweb commented Jul 17, 2019

Hello! after having done the update of joomla 3.7.5 to 3.9.10 I received this error: [17-Jul-2019 13:28:13 Etc/GMT] PHP Fatal error: Call to undefined method JLanguageHelper::parseIniFile() in /home/pescwj6v/public_html/libraries/src/Language/Language.php on line 803

My site is in pt-BR (portuguese-brazil), and both the site and the administrator are without access, the pages are blank. Does anyone have any solutions to this error?

@ghost
Copy link
Copy Markdown

ghost commented Jul 17, 2019

@pescweb Please ask on Forum as this Repository is for coding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants