? Success
Pull Request for # 6186
Referenced as Related to: # 6211

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
26 Feb 2015

To test, create a basic multilanguage site with 3.4.0. Easiest is to use the functionnality when installing Joomla.
Then edit with phpmyadmin the _languages table in the database and set the access column values to 0.

Display frontend is broken.
Patch and test again.

avatar infograf768 infograf768 - open - 26 Feb 2015
avatar joomla-cms-bot joomla-cms-bot - change - 26 Feb 2015
Labels Added: ?
avatar Bakual
Bakual - comment - 26 Feb 2015

Thanks. Looks fine to me.

avatar mahagr
mahagr - comment - 26 Feb 2015

I would also add sticky notification to admin to allow/force users to fix the database integrity issue so we can restore the new code later.

I don't think that isset() is needed as the variable should be always present..?

avatar Hackwar
Hackwar - comment - 26 Feb 2015

as @mahagr says, the isset() is unnecessary.

avatar infograf768
infograf768 - comment - 26 Feb 2015

in the module we just indeed use if($language->access etc. the isset was not in the original PR implementing the access.
Now looking when it was added.

avatar infograf768
infograf768 - comment - 26 Feb 2015

took off the isset.

avatar infograf768
infograf768 - comment - 26 Feb 2015

I suggest to commit this for 3.4.1 and add, when ready, a postinstall message

RTC

avatar infograf768 infograf768 - change - 26 Feb 2015
Labels Added: ?
avatar infograf768 infograf768 - change - 26 Feb 2015
Labels Added: ?
avatar zero-24
zero-24 - comment - 26 Feb 2015

@infograf768 postinstall can be found here: #6195

avatar infograf768 infograf768 - change - 26 Feb 2015
Labels Removed: ?
avatar zero-24 zero-24 - change - 26 Feb 2015
Easy No Yes
avatar zero-24 zero-24 - change - 26 Feb 2015
Status Pending Ready to Commit
avatar zero-24 zero-24 - change - 26 Feb 2015
Category Multilanguage Plugins
avatar zero-24
zero-24 - comment - 26 Feb 2015

moving RTC to match the RTC label on github


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6194.
avatar zero-24 zero-24 - change - 26 Feb 2015
Rel_Number 0 6186
Relation Type Pull Request for
avatar wilsonge
wilsonge - comment - 27 Feb 2015

JM can you just add a code comment above this line just adding

// @todo: In Joomla 2.5.4 and earlier access wasn't set properly and gives a value of 0

or something similar so in the future we can come back to it and try and apply a longer term fix?

avatar infograf768
infograf768 - comment - 27 Feb 2015

done

avatar wilsonge wilsonge - change - 27 Feb 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-02-27 10:55:28
avatar wilsonge wilsonge - close - 27 Feb 2015
avatar wilsonge wilsonge - reference | - 27 Feb 15
avatar wilsonge wilsonge - merge - 27 Feb 2015
avatar wilsonge wilsonge - close - 27 Feb 2015
avatar Kubik-Rubik Kubik-Rubik - reference | - 27 Feb 15
avatar Bakual
Bakual - comment - 27 Feb 2015

in the future we can come back to it and try and apply a longer term fix?

I'd say we can remove it for 4.0. Because then we can assume everyone has saved the value (especially given we have the postinstall message). Also it allows to make backward incompatible changes.

avatar mbabker
mbabker - comment - 27 Feb 2015

You can't even assume it then, today we're making a fix for a change that was applied in 2.5.4, a release almost 3 years ago and on a previous major release branch. The only 'safe' assumption is that people will find ways to break things :smiley:

avatar Bakual
Bakual - comment - 27 Feb 2015

Sure enough, but at least we are allowed to break it :)

avatar infograf768
infograf768 - comment - 28 Feb 2015

Imho it does not cost much to leave the check (which btw also exists in the switcher module).

Add a Comment

Login with GitHub to post a comment