lml / ost

OpenStax Tutor
Other
7 stars 8 forks source link

bug: can't decrease assignment/test duration to 1 day #10

Closed kjdav closed 12 years ago

kjdav commented 12 years ago

Rather, calendar allows you to decrease event duration to 1 day. But when you hit refresh (or navigate to another page) the event reverts back to the last assignment duration before you changed it to 1 day.

I tested to see if this happens when you decrease an event duration from 7 days to 6 days to 5 days ... to 2 days, and the only time it's an issue is decreasing an event (of any duration) to 1 day. Also not a problem when decreasing event times, say from 7am-10pm to 7am-7:30am.

However, if you change a 2-day event starting at 10 am to a 2-day event starting at 8am, it will revert back to 2-day event starting at 10am. So it doesn't save the new duration or the time if you schedule a 1 day event.

Related bug: when you decrease an event length, the cursor stays in "reduce" mode while on that event, instead of reverting back to mouse. so you have to hit refresh or click on another event to get the normal cursor back.

(was testing under walter and admin and trudy)

kevinburleigh75 commented 12 years ago

This issue appears to be fixed with my recent Timezone Fix (branch: assignment_plan_timezone_fix). It is no longer reproducible.

I suggest that @kjdav verify this and close this bug.

jpslav commented 12 years ago

@kjdav please verify and close if true

On Aug 18, 2012, at 10:52 AM, kevinburleigh75 wrote:

This issue appears to be fixed with my recent Timezone Fix (branch: assignment_plan_timezone_fix). It is no longer reproducible.

— Reply to this email directly or view it on GitHub.

kjdav commented 12 years ago

I just did git pull a few minutes ago and I'm still able to reproduce this in dev. after changing to 1 day and hitting refresh, length reverts to most recent length @kevinburleigh75

jpslav commented 12 years ago

Did you restart your server after pulling?

On Aug 18, 2012, at 12:01 PM, Kim Davenport notifications@github.com wrote:

I just did git pull a few minutes ago and I'm still able to reproduce this in dev. after changing to 1 day and hitting refresh, length reverts to most recent length @kevinburleigh75

— Reply to this email directly or view it on GitHub.

kjdav commented 12 years ago

I did... just did it again to be sure

On Sat, Aug 18, 2012 at 3:09 PM, JP Slavinsky notifications@github.comwrote:

Did you restart your server after pulling?

On Aug 18, 2012, at 12:01 PM, Kim Davenport notifications@github.com wrote:

I just did git pull a few minutes ago and I'm still able to reproduce this in dev. after changing to 1 day and hitting refresh, length reverts to most recent length @kevinburleigh75

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/lml/ost/issues/10#issuecomment-7847137.

kjdav commented 12 years ago

this works