6.0.0-beta1
8/15/25

[#13186] Change in WeatherUnderground API
Summary Change in WeatherUnderground API
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org, mrubinsk (at) horde (dot) org
Requester arjen+horde (at) de-korte (dot) org
Created 05/14/2014 (4111 days ago)
Due
Updated 05/16/2014 (4109 days ago)
Assigned 05/14/2014 (4111 days ago)
Resolved 05/16/2014 (4109 days ago)
Github Issue Link
Github Pull Request
Milestone 2.1.1
Patch No

History
05/16/2014 09:17:04 AM Jan Schneider State ⇒ Resolved
Milestone ⇒ 2.1.1
 
05/16/2014 09:15:51 AM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (master):

commit dd139716abd56292f50bf9ede9147c975f024167
Author: Jan Schneider <jan@horde.org>
Date:   Fri May 16 11:15:07 2014 +0200

     [mjr] Add cardinal direction abbreviations (Bug #13186).

  framework/Service_Weather/package.xml |   18 ++++++++++++++----
  1 files changed, 14 insertions(+), 4 deletions(-)

http://github.com/horde/horde/commit/dd139716abd56292f50bf9ede9147c975f024167
05/16/2014 09:15:48 AM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit 3fdea08d96954d78bd17ee0d198543e96cf8e058
Author: Jan Schneider <jan@horde.org>
Date:   Fri May 16 11:14:10 2014 +0200

     Update translations (Bug #13186).

  .../locale/de/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 12010 -> 
11985 bytes
  .../locale/de/LC_MESSAGES/Horde_Service_Weather.po |   18 +++++-----
  .../en_US/LC_MESSAGES/Horde_Service_Weather.po     |   34 
--------------------
  .../locale/es/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 12373 -> 
12346 bytes
  .../locale/es/LC_MESSAGES/Horde_Service_Weather.po |   18 +++++-----
  .../locale/eu/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 868 -> 833 bytes
  .../locale/eu/LC_MESSAGES/Horde_Service_Weather.po |   16 +++++-----
  .../locale/fi/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 12742 -> 
12707 bytes
  .../locale/fi/LC_MESSAGES/Horde_Service_Weather.po |   16 +++++-----
  .../locale/fr/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 11946 -> 
11923 bytes
  .../locale/fr/LC_MESSAGES/Horde_Service_Weather.po |   16 +++++-----
  .../locale/hu/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 11556 -> 
11528 bytes
  .../locale/hu/LC_MESSAGES/Horde_Service_Weather.po |   20 ++++++------
  .../locale/ja/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 9872 -> 9858 bytes
  .../locale/ja/LC_MESSAGES/Horde_Service_Weather.po |   10 +++---
  .../locale/nl/LC_MESSAGES/Horde_Service_Weather.mo |  Bin 11884 -> 
11870 bytes
  .../locale/nl/LC_MESSAGES/Horde_Service_Weather.po |    8 ++--
  17 files changed, 61 insertions(+), 95 deletions(-)

http://github.com/horde/horde/commit/3fdea08d96954d78bd17ee0d198543e96cf8e058
05/14/2014 10:47:12 PM Michael Rubinsky Comment #4
Assigned to Jan Schneider
Assigned to Michael Rubinsky
State ⇒ Assigned
Reply to this comment
Jan, I added the abbreviated strings to 
WeatherUnderground_Strings.php, but I'm not sure what to do with the 
translation stuff in the patch. Can you take a look?
05/14/2014 10:46:04 PM Michael Rubinsky Queue ⇒ Horde Framework Packages
Version ⇒ Git master
 
05/14/2014 10:45:05 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 467f6d6805b25576df7e332d3fa3a8e219479fcd
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Wed May 14 18:43:13 2014 -0400

     Bug: 13186 Add cardinal direction abbreviations.

  .../Service/Weather/WeatherUnderground_Strings.php |    4 ++++
  1 files changed, 4 insertions(+), 0 deletions(-)

http://github.com/horde/horde/commit/467f6d6805b25576df7e332d3fa3a8e219479fcd
05/14/2014 08:04:21 PM arjen+horde (at) de-korte (dot) org Comment #2
New Attachment: Service_Weather.patch Download
Reply to this comment
Attached a patch to fix this
05/14/2014 02:33:34 PM arjen+horde (at) de-korte (dot) org Comment #1
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Queue ⇒ Timeobjects
Summary ⇒ Change in WeatherUnderground API
Type ⇒ Bug
Priority ⇒ 1. Low
Reply to this comment
It looks like WeatherUnderground has changed some responses in their 
API (old -> new):

North -> N
East -> E
South -> S
West -> W

As a result, the translations of these wind directions no longer work.

Saved Queries