Calendar "In office" Bugs

OpenEMR Development
1

medfetch wrote on Saturday, September 10, 2016:

Using the latest codebase on a blank installation:

  1. If I create an “In Office” for a Provider at 10:00AM, the Calendar shows “In Office” at the start of the day’s Calendar, not 10AM as it used to. The duration is also off on the display, no matter if you are displaying by Day or Week.

Putting an OUT of office at 12 noon and the Find Available appointment does work correctly - appts from 10-11:45 are available (15 min interval set).

  1. If two patients are scheduled in the same time slot, changing the appointment status of one triggers the “appointment slot is already in use” pop-up. I don’t think it should - that should only happen when adding an appointment.

  2. The column for the time is twice as wide as it used to be.

Ray

2

medfetch wrote on Saturday, September 10, 2016:

For 1. I see ajax_calendar.css line 281 or so has
.event_in {
background-color: white;
z-index: 1;
white-space: nowrap;
width: 100%;
overflow: hidden;
text-overflow: ellipsis;
border: 1px solid BLACK;
top: 0px !important;
}
Getting rid of the !important removes this problem and puts the IN OFFICE in the correct spot again.

Here is the change:

3

bradymiller wrote on Saturday, September 10, 2016:

Hi,

Sound like the following recent bug fixes may of caused more bugs:


This is also a related calendar bug fix, but don’t think it is a culprit for above reported bugs:

-brady
OpenEMR

4

bradymiller wrote on Saturday, September 10, 2016:

Hi,

I checked in Ray’s above fix into the main codebase since this is a nasty bug. I also re-opened the following PR since part of the original bug discussed in the below PR now still exists:

-brady
OpenEMR

5

bradymiller wrote on Saturday, September 10, 2016:

Hi,

Tracked the timeslot width bug (to wide) to this line of code that was recently added:

This was part of this PR:

I’ll reopen that PR and see if Matrix can come up with a better solution.

-brady
OpenEMR

6

sharonco wrote on Sunday, September 11, 2016:

We will check it today!