Tickets :: [#11005] Disabled uncofigured weather provider spams log with errors.

6.0.0-beta6

4/9/26

Summary	Disabled uncofigured weather provider spams log with errors.
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	rammer (at) ipi (dot) fi
Created	2/17/12 (5165 days ago)
Due
Updated	2/18/12 (5164 days ago)
Assigned	2/17/12 (5165 days ago)
Resolved	2/18/12 (5164 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

02/18/2012 07:33:48 AM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (develop): commit cbf9dc570aba5d2caaf1b25bb1cea3001c20b369 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Sat Feb 18 02:12:36 2012 -0500 Check for a configured provider while ensure()ing the driver. ~~Bug: 11005~~ timeobjects/lib/Driver/Weather.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/cbf9dc570aba5d2caaf1b25bb1cea3001c20b369

02/18/2012 07:33:30 AM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (develop): commit a5ebfcf2c449e4971ee015b618ee2bc8ea7fabb8 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Sat Feb 18 02:07:18 2012 -0500 Throw exception if no weather service configured when trying to obtain a weather driver from the injector. ~~Bug: 11005~~ framework/Core/lib/Horde/Core/Factory/Weather.php \| 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/a5ebfcf2c449e4971ee015b618ee2bc8ea7fabb8

02/18/2012 07:17:05 AM	Michael Rubinsky	Comment #5 State ⇒ Resolved	Reply to this comment
Probably coming from timeobjects. Fixed.

02/18/2012 07:16:41 AM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (master): commit cbf9dc570aba5d2caaf1b25bb1cea3001c20b369 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Sat Feb 18 02:12:36 2012 -0500 Check for a configured provider while ensure()ing the driver. ~~Bug: 11005~~ timeobjects/lib/Driver/Weather.php \| 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/cbf9dc570aba5d2caaf1b25bb1cea3001c20b369

02/18/2012 07:16:35 AM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit a5ebfcf2c449e4971ee015b618ee2bc8ea7fabb8 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Sat Feb 18 02:07:18 2012 -0500 Throw exception if no weather service configured when trying to obtain a weather driver from the injector. ~~Bug: 11005~~ framework/Core/lib/Horde/Core/Factory/Weather.php \| 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/a5ebfcf2c449e4971ee015b618ee2bc8ea7fabb8

02/17/2012 05:13:04 PM	Michael Rubinsky	Comment #2 State ⇒ Feedback Assigned to Michael Rubinsky	Reply to this comment
Where is this being called from? If there is no weather provider configured, we should not be calling the factory to begin with.

02/17/2012 08:11:16 AM	rammer (at) ipi (dot) fi	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Disabled uncofigured weather provider spams log with errors. Queue ⇒ Horde Framework Packages Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Error messages like this are printed in the log every few minutes. HORDE: [timeobjects] PHP ERROR: Undefined index: params [pid 1650 on line 18 of "/usr/share/php/Horde/Core/Factory/Weather.php"] It tries to get a configuration parameter that is not set because the weather provider is not configured.