Needs review
Project:
Yahoo Weather Forecast
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
26 Jan 2012 at 05:47 UTC
Updated:
18 Apr 2013 at 16:57 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
lomo commentedChanged 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). :-)
Comment #2
JimmyAx commentedI have successfully ported the module to Drupal 7. Should be working fine.
http://drupal.org/sandbox/jimmyax/1599268
Comment #3
Balbo commentedAnyone can test the JimmyAx port?
Comment #4
JimmyAx commentedI have it on one of my sites and I haven't experienced any issues with it.
Comment #5
attila.fekete commentedI have ported this module to D7, including support to set up multiple weather blocks. Patch attached.
Comment #6
attila.fekete commentedHere is a compressed version, if you'd like to omit patching :)
Comment #7
lomo commentedWhen 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. :-/
Comment #8
plachI was not able to apply #5 so I used #6. Here is a diff between #6 and the master branch.
Comment #9
plachBtw, I am afraid this module is abandoned. Anyone wishing to take over ownership so that we can go on with the D7 port?
Comment #10
plachSorry, wrong patch.
Comment #11
heldercor commentedI'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.
Comment #12
ebieymjunior commentedAny news on this?