Closing Fancybox

mdsupport wrote on Thursday, November 14, 2013:

In case anyone is trying to use more recent versions of Fancybox, this patch shows one way of handling both versions transparently.

bradymiller wrote on Sunday, November 17, 2013:

Hi,
Is this relevant for current codebase(ie. should it go into the official codebase? if you think so, then will bring it in)?
thanks,
-brady
OpenEMR

mdsupport wrote on Sunday, November 17, 2013:

Not sure how the project handles changes related to deprecated functions. If you anticipate switching Fancybox versions in future, it is one less fix at that time otherwise it is one redundant ‘if’ evaluation.
It is somewhat similar to php ‘split’->‘expand’ changes we made internally.