No upgrade to WeatherHarvest 9.1.0? Reply to topic

Installed WH 9.1.0 over 9.0.0 and had this link in Admin > Modules: Version9.0.0.0 Upgrade to 9.1.0

Clicked on Upgrade and got:
Fatal error: Call to undefined method WeatherHarvest::upgrade() in /home/mark/public_html_wp/admin/modules/modules.php on line 159

And sure enough I can't find a upgrade path ...

Do I have to uninstall/re-install (and fight the Admin i/f again) or should it just run - i.e. are there no db changes?

Initiated by Bugger (Original Entry) - Comments Here Please.

There are no db changes - the upgrade section is present in cpg_inst.php, albeit empty, so it should have simply continued since it contains "return true;".

Odd that you should encounter such a problem, as I didn't. The same applies to all modules

There are only 3 functions in the 9.1.0 release of WH - I checked the zip again just in case:

function WeatherHarvest()
function install()
function uninstall()

But if there are no db changes then I good to go

For grins I added:

function upgrade() { return true; }

And re-did the upgrade to clear the upload link in Admin > Modules.

Thanks!

hmm, that is odd - I just checked the download and realize that the cpg_inst.php file there is not the one I have in my cvs, which means I've uploaded a zip with the incorrect file.

In fact, what I have done is checked out the module without previously committing the updated file - foolish error.

No problem! Easily done - voice of experience

TortoiseCVS on Windoze is fairly good at bringing that to your attention, but now I'm on Linux (Ubuntu) it's a real problem. Still Meld is a much better file compare, so I end up doing directory-level compares and then commit - from inside Meld!

Well, not the first time I've done that, which is what makes it such a foolish error.

Local cvs is the best thing I ever did - I integrated it within a local DF copy so it presents me with cvs info just the same as on the DF site, saving a lot of command line time.