Planned appt indicators

This forum is for programmers who have questions about the source code.
Post Reply
User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Planned appt indicators

Post by drtech » Mon Oct 27, 2008 5:27 am

Hey Jordan, thanks for modifying and including what i did...some colors seem to be off in some situations with the new logic however over in the business module though...(broken is showing up when it shouldn't be on one of my pts and unscheduled planned appts are showing up with a completed background instead of just plain)...I will look at it sometime today and see if I can fix it soon...
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Planned appt indicators

Post by drtech » Mon Oct 27, 2008 7:29 am

got it fixed, will post an update soon to head SVN...
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Planned appt indicators

Post by drtech » Mon Oct 27, 2008 12:09 pm

Jordan,
OK, i fixed the logic...
I also added some text to the Procedure description to help distinguish, esp for those who don't have or want colors. Let me know what you think.

Thanks
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Planned appt indicators

Post by drtech » Thu Oct 30, 2008 5:54 am

The colors and still wrong on the planned appts, v6.1.7. Today's is showing as a past appt. when it should be today's color. I had it working great with the code in 6.2 head SVN. Also I added the text for clarity that did not carry over. Did we just miss some code or did you want it changed? I was using it in my 6.1 version for a day or so and it seemed to be very nice and clear for everyone. If you think it needs to be different, just let me know what you think, otherwise you may have just missed some spots where I updated code...just copy the whole sub routine over and that should take care of it. I will take a look at the SVN changes and maybe I can see...Thanks!
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

User avatar
jordansparks
Site Admin
Posts: 5744
Joined: Sun Jun 17, 2007 3:59 pm
Location: Salem, Oregon
Contact:

Re: Planned appt indicators

Post by jordansparks » Thu Oct 30, 2008 8:44 am

There. It was just a punctuation error. I rarely "copy a whole subroutine" over because I need to see what's going on and I also don't want to damage stable code. As a rule, that's a dangerous action. But sometimes, I miss something.
Jordan Sparks, DMD
http://www.opendental.com

User avatar
drtech
Posts: 1649
Joined: Wed Jun 20, 2007 8:44 am
Location: Springfield, MO
Contact:

Re: Planned appt indicators

Post by drtech » Thu Oct 30, 2008 10:25 am

great...yeah, i thought you probably wanted to look it over and put it in line by line, so I figured that is what happened.
Thanks..i will try it out!
David Fuchs
Dentist - Springfield, MO
Smile Dental http://www.887-smile.com

Post Reply