? Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
11 Jun 2019

Pull Request for Issue #25176

Summary of Changes

Tries to fix user table if json got corrupted.

@alikon

avatar HLeithner HLeithner - open - 11 Jun 2019
avatar HLeithner HLeithner - change - 11 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Jun 2019
Category SQL Administration com_admin Postgresql MS SQL
avatar richard67
richard67 - comment - 11 Jun 2019

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.

avatar mbabker
mbabker - comment - 11 Jun 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.

avatar HLeithner HLeithner - change - 11 Jun 2019
Labels Added: ?
avatar richard67
richard67 - comment - 11 Jun 2019

@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?

avatar HLeithner
HLeithner - comment - 11 Jun 2019

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

avatar alikon
alikon - comment - 11 Jun 2019

yes make sense to remove the buggy replace imo

avatar mbabker
mbabker - comment - 11 Jun 2019

Should be good now.

avatar richard67
richard67 - comment - 11 Jun 2019

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

avatar SniperSister
SniperSister - comment - 11 Jun 2019

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

avatar HLeithner
HLeithner - comment - 11 Jun 2019

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...

avatar richard67
richard67 - comment - 11 Jun 2019

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 ?

avatar HLeithner
HLeithner - comment - 11 Jun 2019

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

avatar mbabker
mbabker - comment - 11 Jun 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
avatar richard67
richard67 - comment - 11 Jun 2019

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

avatar alikon
alikon - comment - 11 Jun 2019

lgtm

avatar HLeithner
HLeithner - comment - 11 Jun 2019

I added the comment.

avatar richard67
richard67 - comment - 11 Jun 2019

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.

avatar richard67
richard67 - comment - 11 Jun 2019

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.

avatar richard67 richard67 - test_item - 11 Jun 2019 - Tested successfully
avatar alikon
alikon - comment - 11 Jun 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.

avatar alikon alikon - test_item - 11 Jun 2019 - Tested successfully
avatar HLeithner HLeithner - change - 11 Jun 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-06-11 19:52:17
Closed_By HLeithner
avatar HLeithner HLeithner - close - 11 Jun 2019
avatar HLeithner HLeithner - merge - 11 Jun 2019
avatar HLeithner
HLeithner - comment - 11 Jun 2019

thx for the help, release is out.

avatar pescweb
pescweb - comment - 17 Jul 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?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Jul 2019

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

Add a Comment

Login with GitHub to post a comment