6.0.0-beta1
7/6/25

[#10854] Dewpoint of 0 degrees not reported
Summary Dewpoint of 0 degrees not reported
Queue Horde Base
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester arjen+horde (at) de-korte (dot) org
Created 12/11/2011 (4956 days ago)
Due
Updated 12/14/2011 (4953 days ago)
Assigned 12/12/2011 (4955 days ago)
Resolved 12/14/2011 (4953 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
12/14/2011 02:56:09 PM Michael Rubinsky Comment #4
State ⇒ Resolved
Reply to this comment
Committed, thanks!
12/14/2011 02:55:50 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git for this ticket:

Bug: 10854 Fix displaying of dewpoint.
Don't display title if it's not present, and fix displaying
when it is zero.
Signed-off-by: Michael J Rubinsky <mrubinsk@horde.org>

  1 files changed, 2 insertions(+), 4 deletions(-)
http://git.horde.org/horde-git/-/commit/a53738cd111aefbea11c3497e524bb4c40de1c7a
12/12/2011 09:59:49 AM Jan Schneider Assigned to Michael Rubinsky
State ⇒ Assigned
 
12/11/2011 05:48:53 PM arjen+horde (at) de-korte (dot) org Comment #2 Reply to this comment
In retrospect, since we're expecting a numerical value in the first 
place (given the 'round'), it is probably sufficient to just use

+        if (is_numeric($current->dewpoint)) {

here.
12/11/2011 12:28:29 PM arjen+horde (at) de-korte (dot) org Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Dewpoint of 0 degrees not reported
Queue ⇒ Horde Base
Milestone ⇒
Patch ⇒ Yes
New Attachment: Block_Weather.diff Download
State ⇒ Unconfirmed
Reply to this comment
When using metric units, a dewpoint of 0 degrees centigrade is not 
reported. The attached patch fixes this, by checking if 
$current->dewpoint is either null (or 0) or a numeric value. It also 
fixes a PHP error because $dewpoint is not defined.

Saved Queries