Comments

lomo’s picture

Title: Upgrade to Drupal 7? » [Yahoo Weather Forecast] D7 version?
Assigned: Unassigned » lomo

Changed title so that it's clear what project this refers to when we see new activity in our dashboards.

I've also set the "assigned" to myself as I'm working on a patch which I hope to get working and upload here (soon). :-)

JimmyAx’s picture

Assigned: lomo » Unassigned
Status: Active » Needs review

I have successfully ported the module to Drupal 7. Should be working fine.

http://drupal.org/sandbox/jimmyax/1599268

Balbo’s picture

Anyone can test the JimmyAx port?

JimmyAx’s picture

I have it on one of my sites and I haven't experienced any issues with it.

attila.fekete’s picture

I have ported this module to D7, including support to set up multiple weather blocks. Patch attached.

attila.fekete’s picture

StatusFileSize
new644.42 KB

Here is a compressed version, if you'd like to omit patching :)

lomo’s picture

When I attempted my port, I didn't quite get it working, but I think I made some improvements here and there that might be mergeable into a clean port, so I'll see if I have time to see if there is anything worth salvaging from the day I spent working on a port. Sorry I never followed up here about that and uploaded my work since I think I was close enough someone else might have been able to fix the remaining issues faster than duplicating all my work... this just slipped through the cracks. :-/

plach’s picture

I was not able to apply #5 so I used #6. Here is a diff between #6 and the master branch.

plach’s picture

Btw, I am afraid this module is abandoned. Anyone wishing to take over ownership so that we can go on with the D7 port?

plach’s picture

Sorry, wrong patch.

heldercor’s picture

I've used #6 but it assumed a fresh install.

If anyone needs to upgrade their data as well, I've added update functions for that in the attached patch.

I've only included the fields that were renamed, though.

ebieymjunior’s picture

Any news on this?