Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-31913

Backport TinyMCE 3.4.9 to 22 stable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1
    • Fix Version/s: 2.2.3
    • Component/s: HTML Editor (TinyMCE)
    • Labels:
      None
    • Testing Instructions:
      Hide

      pasted from MDL-30636:

      1/ test tinymce in moodle
      2/ test file picker integration - both files and links
      3/ concentrate on Moodle specific plugins - dragmath, moodlemotions, moodlenolink, moodlemedia, spellchecker

      Show
      pasted from MDL-30636 : 1/ test tinymce in moodle 2/ test file picker integration - both files and links 3/ concentrate on Moodle specific plugins - dragmath, moodlemotions, moodlenolink, moodlemedia, spellchecker
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:

      Description

      By MDL-30636, TinyMCE 3.4.9 was upgraded into master (2.3).

      This is about to backport it both to 22 (now using 3.4.6) and 21 (now using 3.4.2) stables (together and with 1w separation) once it's probed to be working ok on master.

      Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            skodak Petr Skoda added a comment -

            To integrators:
            1/ 2.1 already diverged too much and the new editor can not be easily backported there, there is no technical reason to not upgrade from 2.1 to 2.2 so I decided to not spend more time on backporting

            2/ the MDL in commit messages links MDL-30636, I guess it is better that way

            Show
            skodak Petr Skoda added a comment - To integrators: 1/ 2.1 already diverged too much and the new editor can not be easily backported there, there is no technical reason to not upgrade from 2.1 to 2.2 so I decided to not spend more time on backporting 2/ the MDL in commit messages links MDL-30636 , I guess it is better that way
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            1/ oki, under you responsibility
            2/ oki, you are laaaazzzzzyyyy eh? :-P

            Looking to this. Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - 1/ oki, under you responsibility 2/ oki, you are laaaazzzzzyyyy eh? :-P Looking to this. Ciao
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Tidying issue title as far as we are going to backport only to 22.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Tidying issue title as far as we are going to backport only to 22.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks! (22 only)

            Side comment: Is https://github.com/moodle/custom-tinymce missing the 22_STABLE_3.4.9 ?

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks! (22 only) Side comment: Is https://github.com/moodle/custom-tinymce missing the 22_STABLE_3.4.9 ? Ciao
            Hide
            skodak Petr Skoda added a comment -

            yes, I did not create a new branch for custom-tinymce because I backported stuff from master (keeping the original MDL).

            Show
            skodak Petr Skoda added a comment - yes, I did not create a new branch for custom-tinymce because I backported stuff from master (keeping the original MDL).
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys, tested thoroughly and passed

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys, tested thoroughly and passed
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            FCT (fixed, closing, thanks). Ciao

            "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!"
            ~ Benjamin Disraeli

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - FCT (fixed, closing, thanks). Ciao "I feel a very unusual sensation - if it is not indigestion, I think it must be gratitude!" ~ Benjamin Disraeli

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12