? Error

User tests: Successful: Unsuccessful:

avatar elinw
elinw
26 Oct 2013

...$bootstrapSize == 1.

avatar elinw elinw - open - 26 Oct 2013
avatar beat
beat - comment - 26 Oct 2013

Thanks for the suggestion, but it looks like you are redoing this one #2224 that i have closed in favor of this correct fix one: #2231

Suggesting that you close this one and test #2231 instead, as it doesn't have an issue tracker commented here anyway, and is not a correct fix, as #2231 is the correct way to fix the past erroneous sample data, while permitting users to have tiny-width modules in cpanel from 3.2 onwards.

avatar brianteeman
brianteeman - comment - 31 Oct 2013

as #2231 has been committed I am closing this one

avatar brianteeman brianteeman - close - 31 Oct 2013
avatar elinw
elinw - comment - 3 Nov 2013

No and as we know it is still broken. Redoing code ... not clear why that would be said if you look at the code.

avatar elinw elinw - reopen - 3 Nov 2013
avatar beat
beat - comment - 3 Nov 2013

That would mean that #2231 has a bug (i.e. the query isn't executed), which needs to be fixed.

This one is only a workaround.

avatar Bakual
Bakual - comment - 3 Nov 2013

I have just installed Joomla 3.1.5 with english default sample data. Verified that those two modules had the bootstrap_size set to 1.
Then updated to Joomla 3.2.0rc using the Joomla Updater and the cPanel view didn't have any issues at all. The two modules had their bootstrap_size corrected to 0.

So the query works fine and is executed during update, the cPanel view is fixed. I have no clue what Elin is referring to.

avatar beat
beat - comment - 3 Nov 2013

@elinw I do not see a tracker item and no instructions to reproduce this bug. Can you please first reproduce the bug, and then either add a tracker item with instructions to reproduce the bug, or if it works for you, close this PR ?

avatar Bakual
Bakual - comment - 2 Mar 2014

Closing as it's no issue.

avatar Bakual Bakual - close - 2 Mar 2014

Add a Comment

Login with GitHub to post a comment