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

                Attachments

                  Issue Links

                  has a non-specific relationship to

                  Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-32141 outdated TinyMCE in 2.1.x does not work in Firefox 11

                  • Critical - Crashes server, loss of data, severe memory leak
                  • Closed

                  Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-30636 Upgrade to TinyMCE 3.4.9

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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
                    Permalink
                    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