mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #31807
[Bug 1521058] Re: Bad merge cruft in htdocs/js/tinymce/tinymce.jquery.js
The merged code contains a duplicated of the module
"tinymce/dom/StyleSheetLoader", and a partial duplicate of
"tinymce/dom/DOMUTils". After comparing these with what was there before
patch 5ba3418499, I saw some differences, but it looked like the copied
versions were simpler than what was there before, rather than being
purposeful changes.
Comparing it further, it looked like the versions in the merge commit
were from TinyMCE 4.0.16. I think what happened is that I upgrade
TinyMCE to 4.1.9 on the main branch, and then when the front-end people
tried to merge their changes in, they got a merge conflict because their
branch was still on 4.0.16.
That being the case, the best thing is just to revert that section. I've
pushed a patch to do that: https://reviews.mahara.org/5809
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1521058
Title:
Bad merge cruft in htdocs/js/tinymce/tinymce.jquery.js
Status in Mahara:
Confirmed
Status in Mahara 15.10 series:
Confirmed
Status in Mahara 16.04 series:
Confirmed
Bug description:
Robert noticed that there is a merge conflict ">>>>>" line left over
in tinymce.jquery.js. It was apparently introduced in commit
5ba3418499cc419e, one of the bootstrap commits. The merge conflict was
later partially cleaned up in commit e0d6f1c67f8ac, but this part was
overlooked.
Looking closer, it's also apparent that quite a bit of duplicated code
from that merge conflict was also left in place.
It's unclear whether this has caused any bugs, or which section of
TinyMCE would cause this code to be executed.
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1521058/+subscriptions
References