OpenEMR 4.1.2 Patch

bradymiller wrote on Thursday, June 12, 2014:

Hi Ian,

Sounds like it’s going to be a long-term project :slight_smile:
See here for some wise words:
https://sourceforge.net/p/openemr/discussion/202506/thread/e82523fd/?limit=25&page=1#46cd

Regarding php-gacl, that project is dead. We have embedded the project into OpenEMR’s codebase/database along with some fixes/customizations.

Regarding adodb, that would be tough since each version change would require intensive testing; Kevin likely has more to add here.

Regarding phpmyadmin, that is basically to make it easier for diy types. I’ve been a heavy proponent for including that in the past, but it may finally be time to let that one go (because it’s too tough to keep up with security updates on the phpmyadmin stuff).

Is there an advantage to using a symlink rather than just setting apache to point to it (like is done in the fixed not yet released openemr package)?

-brady
OpenEMR

yehster wrote on Thursday, June 12, 2014:

The php5-adodb is not the library for database access. It is a subset of code that is compiled as c for optimization. The PHP files are still needed in order to work.

That Debian package installs this extension, which does not sufficiently handle the adodb dependency OpenEMR needs.
http://adodb.sourceforge.net/#extension

iankarlwallace wrote on Friday, June 13, 2014:

Kevin and Brady - Thanks for the responses. Will make note that the GACL
and adodb stuff need to be left alone.

There’s a nice phpMyAdmin package that installs and configures correctly
that we should probably just point people too if using Linux (phpmyadmin)
or installing it in Windows from the site.

Probably no clear benefit to symlink things should just point to it and not
muddle up the file system.

The problem with putting anything in /var/www/html is that the Debian
Policy says that’s not allowed (it’ll spit out errors when you run lintian
which will prevent it from being accepted to Debian repos - the whole point
of doing this little exercise).

Ok, phpmyadmin will come out, others stay in and keep plugging away at
apache, etc.

Interesting phpmyadmin is using a Apache configuration method similar to
the way you were first doing it (not as a site) … which on second thought
might be a better option. I am pretty much going to replicate it since
it’s so clean.

cheers
ian

On Wed, Jun 11, 2014 at 11:48 PM, Kevin Yeh yehster@users.sf.net wrote:

The php5-adodb is not the library for database access. It is a subset of
code that is compiled as c for optimization. The PHP files are still needed
in order to work.

That Debian package installs this extension, which does not sufficiently
handle the adodb dependency OpenEMR needs.
ADOdb download | SourceForge.net

OpenEMR 4.1.2 Patch
https://sourceforge.net/p/openemr/discussion/202506/thread/3a8445be/?limit=25&page=1#c2f7/18ea/0cc1/fe5e/9d5d/c7af

Sent from sourceforge.net because you indicated interest in
OpenEMR / Discussion / Developers

To unsubscribe from further messages, please visit
SourceForge.net: Log In to SourceForge.net


Ian Wallace - CCRMC DFM Staff Physician - (c) 303.681.5732

fr4nkie wrote on Friday, June 13, 2014:

Dr. Lee,

My Out events are still orange after patching. Are there any other steps you took to revert back to the selected color in Administration > Other > Calendar > Categories? I tried to clear smarty cache and clear local browser cache with no change.

fsgl wrote on Friday, June 13, 2014:

Hi Frankie,

Go to openemr/interface/themes/ajax_calendar.css
Line 339 (the very last line)
background-color:#

Pick a hexadecimal triplet from here with a number of darker and lighter shades for each color.

It may be necessary to clear the cache after replacing the pumpkin color in the .css file.

fr4nkie wrote on Friday, June 13, 2014:

Perfect. Just what I was looking for. Thank you! :slight_smile:

fsgl wrote on Friday, June 13, 2014:

My pleasure.

bradymiller wrote on Friday, June 13, 2014:

Hi Ian,

It seems like the real elephant in the room here is the /var/www (or /var/www/html) paths. It would be useful to know how strict this requirement is and whether it can be overrided; considering it is a lintian Error rather than a Warning guessing not much leniency, but you never know. If it does need to move, then makes sense to see what phpmyadmin does (I am guessing phpmyadmin does not separate it’s writable config files from the read-only, so perhaps it would only involve migrating it during an upgrade; migrate it in preinst maintainer script and then the rest happens automatically).

Once that is out of the way, then seems like just need to clean up the codebase some more to deal with the left over random lintian stuff. In the maintainer scripts, I am also confident that the direct modification of the php.ini file will need to go (phpmyadmin does a cool thing where it overrides php settings in the apache site config file, if needed).

-brady
OpenEMR

fsgl wrote on Thursday, June 19, 2014:

Brady was probably too sleepy to announce the release of the latest patch.

Patch 7 was released today.

iankarlwallace wrote on Thursday, June 19, 2014:

Brady - Sorry work/family have gotten in the way of keeping up with
emails. As you suspect there is no bending on the lintian Error’s.
Warnings I think most will let slide … but only if we are willing to look
into fixing them. I agree with most of the error messages anyway. I have
a package that has most of the Error’s taken care of … but your nicely
written upgrade scripts/etc need to be re-worked at some point - lintian
complains about the templates, debconf stuff and something about debhelper
tags that we are missing.

I am first focused on just cleaning up the structure and getting ride of
the extra license files that needed to be merged to make it happy.

cheers
ian

On Fri, Jun 13, 2014 at 12:30 AM, Brady Miller bradymiller@users.sf.net
wrote:

Hi Ian,

It seems like the real elephant in the room here is the /var/www (or
/var/www/html) paths. It would be useful to know how strict this
requirement is and whether it can be overrided; considering it is a lintian
Error rather than a Warning guessing not much leniency, but you never know.
If it does need to move, then makes sense to see what phpmyadmin does (I am
guessing phpmyadmin does not separate it’s writable config files from the
read-only, so perhaps it would only involve migrating it during an upgrade;
migrate it in preinst maintainer script and then the rest happens
automatically).

Once that is out of the way, then seems like just need to clean up the
codebase some more to deal with the left over random lintian stuff. In the
maintainer scripts, I am also confident that the direct modification of the
php.ini file will need to go (phpmyadmin does a cool thing where it
overrides php settings in the apache site config file, if needed).

-brady
OpenEMR http://www.open-emr.org/

OpenEMR 4.1.2 Patch
https://sourceforge.net/p/openemr/discussion/202506/thread/3a8445be/?limit=25&page=1#c2f7/18ea/0cc1/fe5e/9d5d/c7af/6be2/a1df

Sent from sourceforge.net because you indicated interest in
OpenEMR / Discussion / Developers

To unsubscribe from further messages, please visit
SourceForge.net: Log In to SourceForge.net


Ian Wallace - CCRMC DFM Staff Physician - (c) 303.681.5732

cmswest wrote on Saturday, June 21, 2014:

thank you Brady and thanks for your tireless posting fsgl!

hope everyone enjoys the last night of spring for those in the northern hemisphere, happy borealis australis you true southerners!

winter antarctica

fsgl wrote on Saturday, June 21, 2014:

Happy Summer Solstice & have fun hiking, Stephen.

Lovely images.

pfindan wrote on Thursday, August 07, 2014:

Hello everyone! Regarding the bug about the ‘OUT’ event splitting the previous event, changing it’s duration from 0 minutes to 15 minutes fixes this problem. I’ll send the patch in when I have time.