? Success

User tests: Successful: Unsuccessful:

avatar Paladin
Paladin
23 Feb 2015

"Nothing to Update" is not an error condition, it's a valid result.

No real BC issue, here. Those using the old workaround for this fall will be unaffected, while any others can either put a blank sql file in, or omit the sql file when there are no schema changes.

avatar Paladin Paladin - open - 23 Feb 2015
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2015
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2015
Labels Added: ?
avatar wilsonge wilsonge - change - 24 Feb 2015
Milestone Added:
avatar zero-24 zero-24 - change - 24 Feb 2015
Category Administration Updating
avatar wilsonge wilsonge - change - 18 Mar 2015
Milestone Added:
avatar wilsonge wilsonge - change - 18 Mar 2015
Milestone Removed:
avatar brianteeman
brianteeman - comment - 24 May 2015

Please provide instructions how someone can test this, otherwise it will just site here for enough 3 months without comment


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

avatar Paladin
Paladin - comment - 2 Jun 2015

Instructions on how to test it? Details on how it arises can be found in the related issue I filed prior to writing the fix. (#6157)

As a manual test, install an extension (and to be thorough, update it at least once) that does not have a schema update present in, nor a provision for one to have ever happened. It will probably have to be a custom-made one for this purpose, as the custom in extensions to this point has been to always provide a schema update that's basically a no-op (does nothing) as a workaround to the bug (don't get me started on workarounds vs fixes). If an extension that has never had a schema update installs correctly and updates code once without throwing an error, it passes.

avatar brianteeman
brianteeman - comment - 2 Jun 2015

Thanks for the update and for adding the link to the original issue. However it is still pretty hard for anyone to test without and extension that matches your conditions. Do you have one?

avatar Paladin
Paladin - comment - 2 Jun 2015

It's a specialized bug you will likely only encounter in the early stages of creating an extension (component more likely than any other one). I can probably sanitize one of mine (remove the proprietary info from it, as it's custom-built for a particular site rather than generic) but it might be just as easy to take an early version of an existing one (that only has the empty schema update) and simply delete the do-nothing schema update file. (You might even be able to use the hello world component from the tutorials for it, as it documents including it.)

I'll do some rummaging around when I get home; I can probably locate or reproduce an early draft of the component I was working on for a JCM article before things got hectic around here. It'll suck as a component, but it should put the bug on display.

avatar brianteeman
brianteeman - comment - 2 Jun 2015

Would be great if you could. The fewer hoops people have to jump through to test an issue the more likely it will be that they will test it

avatar wilsonge
wilsonge - comment - 17 Jun 2015

@Paladin I agree with this fix but giving us a test component will definitely help get you testers

avatar wilsonge wilsonge - change - 17 Jun 2015
Milestone Added:
avatar wilsonge wilsonge - change - 17 Jun 2015
Milestone Removed:
avatar zero-24 zero-24 - change - 2 Jul 2015
Milestone Removed:
avatar zero-24 zero-24 - change - 2 Jul 2015
Milestone Removed:
avatar Paladin
Paladin - comment - 4 Jul 2015

I know. But unfortunately the one that surfaced this problem has long since progressed past that point and I’ve been hit with a “perfect storm” of emergency tasks that need done, not the least of which being a web host that just packed up and left town with several of my servers, and most of my DNS information, pre-empting my time rather drastically. Either use the description I gave on how to duplicate this issue, or be patient and I’ll dig something out when the dust settles.

On Jun 17, 2015, at 3:39 PM, George Wilson > wrote:

@Paladin https://github.com/Paladin I agree with this fix but giving us a test component will definitely help get you testers


Reply to this email directly or view it on GitHub #6156 (comment).

avatar wilsonge wilsonge - change - 8 May 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-08 16:21:03
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 May 2016
avatar wilsonge wilsonge - merge - 8 May 2016
avatar wilsonge wilsonge - reference | 87c64a5 - 8 May 16
avatar wilsonge wilsonge - merge - 8 May 2016
avatar wilsonge wilsonge - close - 8 May 2016
avatar wilsonge wilsonge - change - 8 May 2016
Milestone Added:
avatar wilsonge
wilsonge - comment - 8 May 2016

Merged on review

Add a Comment

Login with GitHub to post a comment