Exclusive eBooks, Software Solutions, Quality Guaranteed
 
Web NukeBiz
Welcome to NukeBiz · Members
Amazon
show cart or checkout0 items
Cart Value: 0.00

Forums » DragonflyCMS Projects » Weather Harvest » No upgrade to WeatherHarvest 9.1.0?
No upgrade to WeatherHarvest 9.1.0?
Port of IBdeeming's nuke module
Post new topic    Reply to topic    Printer Friendly Page     Forum Index »  Miscellania » Weather Harvest

View previous topic :: View next topic  
Author Message
layingback
Nice Poster
Nice Poster

Offline Offline
Joined: Feb 26, 2007
Posts: 77

PostPosted: Wed May 14, 2008 2:09 pm
Post subject: No upgrade to WeatherHarvest 9.1.0?

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.


layingback's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Linux/2.2.8/5.1.73/5.4.21/9.4.0
Back to top
View user's profile Visit poster's website
Phoenix
Chairman
Chairman

Offline Offline
Joined: Mar 18, 2004
Posts: 1532
Location: Netizen
PostPosted: Wed May 14, 2008 3:40 pm
Post subject: Re: No upgrade to WeatherHarvest 9.1.0?

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 Confused

_________________
DonationsPro for vBulletin, SMF, MyBB, DragonflyCMS


Phoenix's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Back to top
View user's profile Visit poster's website ICQ Number Yahoo Messenger Photo Gallery
layingback
Nice Poster
Nice Poster

Offline Offline
Joined: Feb 26, 2007
Posts: 77

PostPosted: Fri May 16, 2008 9:33 am
Post subject: Re: No upgrade to WeatherHarvest 9.1.0?

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 Wink

For grins I added:

Code::
    function upgrade() {

    	return true;

    }


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

Thanks!


layingback's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Linux/2.2.8/5.1.73/5.4.21/9.4.0
Back to top
View user's profile Visit poster's website
Phoenix
Chairman
Chairman

Offline Offline
Joined: Mar 18, 2004
Posts: 1532
Location: Netizen
PostPosted: Fri May 16, 2008 10:02 am
Post subject: Re: No upgrade to WeatherHarvest 9.1.0?

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.

_________________
DonationsPro for vBulletin, SMF, MyBB, DragonflyCMS


Phoenix's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Back to top
View user's profile Visit poster's website ICQ Number Yahoo Messenger Photo Gallery
layingback
Nice Poster
Nice Poster

Offline Offline
Joined: Feb 26, 2007
Posts: 77

PostPosted: Fri May 16, 2008 10:10 am
Post subject: Re: No upgrade to WeatherHarvest 9.1.0?

No problem! Easily done - voice of experience Wink

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!


layingback's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Linux/2.2.8/5.1.73/5.4.21/9.4.0
Back to top
View user's profile Visit poster's website
Phoenix
Chairman
Chairman

Offline Offline
Joined: Mar 18, 2004
Posts: 1532
Location: Netizen
PostPosted: Fri May 16, 2008 10:36 am
Post subject: Re: No upgrade to WeatherHarvest 9.1.0?

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.

_________________
DonationsPro for vBulletin, SMF, MyBB, DragonflyCMS


Phoenix's server specs (Server OS / Apache / MySQL / PHP / DragonflyCMS)
Back to top
View user's profile Visit poster's website ICQ Number Yahoo Messenger Photo Gallery
Display posts from previous:   
Post new topic    Reply to topic    Printer Friendly Page    Forum Index »  Miscellania » Weather Harvest
Page 1 of 1
All times are GMT + 9.5 Hours



Jump to:  


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You cannot download files in this forum