Posted by chriscohen on September 26, 2008 at 1:25pm
Jump to:
| Project: | Weather |
| Version: | 6.x-5.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | toddy |
| Status: | closed (fixed) |
Issue Summary
I was recently required to do some work on the weather module to provide functionality on a client site, but managed to develop the changes in such a way that I believe they will be able to be used on any Drupal site, so I am submitting this patch in case it's of use.
- Adds a weather search with autocomplete at the /weather path.
- URLs like /weather/foo will attempt to find a corresponding ICAO code and show the weather for this, or search for a location or country if there is no exact match.
- Weather search result is displayed using the existing
theme_weather()function so that themes can freely override it.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| weather-frontend.patch | 7.78 KB | Ignored: Check issue status. | None | None |
Comments
#1
Hi,
thanks a lot for sharing this patch. I like it very much and will probably integrate it soon. However, there are still some things to be done, e.g. formatting the code according to http://drupal.org/coding-standards.
Regards,
Tobias
#2
I've updated your patch and committed this to CVS. Thanks a lot!
Regards,
Tobias
#3
Automatically closed -- issue fixed for two weeks with no activity.