Incorrect display of Equation of Time when seconds activated and -1<time<0

Bug #1406573 reported by Jean-Michel Vourgère
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Stellarium
Fix Released
Low
gzotti
stellarium (Debian)
New
Undecided
Unassigned

Bug Description

Hi!

I enabled plugin "Equation of time". There is a small issue when time is between -1 minute and 0. The minus signed is not displayed.

For example, being in Paris, and selecting 2014-12-25, I have this display:
"Equation of time: 0m07s" while it should be "Equation of time: -0m07s".
If I disable "Use minutes and seconds" in the plug in configuration, I have the correct display: "Equation of time: -0.12m".

I traced the bug to line 137 of plugins/EquationOfTime/src/EquationOfTime.cpp

timeText = QString("%1: %2%3%4%5").arg(messageEquation, QString::number((int)time), messageEquationMinutes, messageSecondsValue, messageEquationSeconds);

The problem is that time is 0, but we should actually print "-0" because the number of second - displayed just after - is negative.

I'm using Debian Jessie. Version is 0.13.1-1.

I'd like to take that opportunity to thank you for that fantastic piece of code. I used it hours, I love it. Thank you so much!

description: updated
description: updated
Revision history for this message
gzotti (georg-zotti) wrote :

fixed in r7254

Changed in stellarium:
status: New → Confirmed
assignee: nobody → gzotti (georg-zotti)
milestone: none → 0.13.2
importance: Undecided → Low
status: Confirmed → Fix Committed
gzotti (georg-zotti)
summary: - Incorrect display of ΔT when seconds activated and -1<time<0
+ Incorrect display of Equation of Time when seconds activated and
+ -1<time<0
Changed in stellarium:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.