Opened 17 years ago

Closed 15 years ago

Last modified 15 years ago

#357 closed Defect (fixed)

Advanced Preferences can't handle "24:00"

Reported by: Didactylos Owned by: Didactylos
Priority: Minor Milestone: Undetermined
Component: Manager Version:
Keywords: manager advanced-preferences Cc:

Description (last modified by Nicolas)

Advanced Preferences can't handle the value "24:00". Neither can the simple prefs, but that doesn't matter since "Never" isn't a useful value unless you can override it with the per-day settings.

(This is almost-but-not-quite a duplicate of #300)

But this is just another bug in the advanced preferences. If it were my call, I'd bump the entire advanced preferences dialog to 6.0.

Also see #351, #352 and #173.

Change History (9)

comment:1 Changed 17 years ago by Nicolas

Description: modified (diff)

Just a sidenote: you can link to the advanced_preferences tag to see all the related tickets.

comment:2 in reply to:  description ; Changed 17 years ago by Ageless

Replying to Didactylos:

Advanced Preferences can't handle the value "24:00".

If you mean in the time slots, then that's normal, isn't it? Digital clocks don't show that it is 24:00 either at midnight. It changes from 23:59 to 0:00.

So the entries go from 0:00 to 23:59.

comment:3 in reply to:  2 Changed 17 years ago by Didactylos

Replying to Ageless: Almost correct. Midnight is 00:00 and 24:00 - although since clocks can't display two values at once, nearly all of them show 00:00. But BOINC treats 24:00 as exactly what it represents; midnight at the end of the day. As you can see, using this value is the only way to specify that BOINC never compute or connect. Here's the actual code:

    if (start==end) return false;
    if (start==0 && end==24) return false;
    if (start==24 && end==0) return true;
    if (start < end) {
        return (hour < start || hour > end);
    } else {
        return (hour >= end && hour < start);
    }

comment:4 Changed 17 years ago by Didactylos

I think this is bumped to milestone:6.0 now.

comment:5 Changed 17 years ago by Nicolas

Milestone: 5.106.0

Bumped.

comment:6 Changed 16 years ago by Didactylos

Keywords: advanced-preferences added; advanced_preferences removed
Owner: changed from romw to Didactylos
Status: newassigned

Combined #300 with this ticket.

comment:7 Changed 15 years ago by romw

Milestone: 6.6Undetermined

comment:8 Changed 15 years ago by davea

Resolution: fixed
Status: assignedclosed

(In [18866]) - manager: allow 24:00 to be used in advanced prefs dialog

(from Ian Hay; fixes #357)

comment:9 Changed 15 years ago by romw

(In [18909]) - manager: allow 24:00 to be used in advanced prefs dialog

(from Ian Hay; fixes #357)

clientgui/

DlgAdvPreferences?.cpp

Note: See TracTickets for help on using tickets.