lh-ehr
lh-ehr copied to clipboard
GSoC 2018 : Calendar Improvements-Repeat parameters in Calendar Admin
Fixes #767
Repeat parameters are added in Calendar Admin so that a set of repeat parameters settings can be stored with respect to an existing category or new category and can be accessed as default settings of repeat parameters corresponding to the selected category in patient or provider event panel.
Preview:
Work done:
- Added repeat parameters in calendar admin (admin.php) as they were in event panel (add_edit_event.php).
- Made some changes in admin.php and admin_helper.php so that they store and display settings as selected by user corresponding to a category in table libreehr_postcalendar_categories.
- Made some changes in add_edit_event.php and appointments.inc.php so that event panel displays default settings in repeat parameters (from table libreehr_postcalendar_categories) corresponding to a selected category in case of a new event. If it is an existing event, repeat parameters settings will be displayed according to table libreehr_postcalendar_events, which is already the case.
@teryhill Please have a look.
@teryhill Added a few things to fix some issues. Expected behavior:
- In case of new events, set recurring parameters according to values set in admin upon opening of event panel (default category) or when category changes.
- In case of existing events, set recurring parameters according to values set in table libreehr_postcalendar_events upon opening of event panel (selected category) which is already the case but set recurring parameters according to values set in admin when category changes.
@tmccormi can you see if this is the way you want this to work?
OK. Looked at this, and it seems to:
- Work without crashing.
- Solve the use-case outlined by htuck.
@aethelwulffe added license headers in last commit.
I have not been looking at the attribution on files lately. Ooops. Good catch. -Folks, this is ready for further review.
@tmccormi Please have a look.
@teryhill @tmccormi @Trodrige @pri2si17-1997 @Perseus and everyone else. Please check this.
Fixed conflicts. Second test by someone else please!
Apparently there is an issue here.