Details

    • Type: Task Task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.2
    • Component/s: Libraries
    • Labels:
      None

      Description

      overlib was used by some question type in 2.0, since 2.1 it is not used anybody in official distribution - yay!

        Gliffy Diagrams

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment - - edited

          Uhm, I've found some uses in contrib:

          • patches/LSU_gradebook
          • patches/postnuke_moodle
          • plugins/mod/dimdim
          • plugins/mod/game
          • plugins/mod/sclipowebclass
          • plugins/mod/webconference

          Plus 2 references in core @ lib/smarty/plugins (although seems they are unused)

          Do we need to do anything with those smarty uses? I guess contrib ones can be passed with proper documentation @ http://docs.moodle.org/dev/Moodle_2.2_release_notes

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - - edited Uhm, I've found some uses in contrib: patches/LSU_gradebook patches/postnuke_moodle plugins/mod/dimdim plugins/mod/game plugins/mod/sclipowebclass plugins/mod/webconference Plus 2 references in core @ lib/smarty/plugins (although seems they are unused) Do we need to do anything with those smarty uses? I guess contrib ones can be passed with proper documentation @ http://docs.moodle.org/dev/Moodle_2.2_release_notes Ciao
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Oki, going ahead with this.

          Some day we should also remove Smarty itself, after a quick look I only found "question/format/qti_two" using it. Should we create an issue about that?

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Oki, going ahead with this. Some day we should also remove Smarty itself, after a quick look I only found "question/format/qti_two" using it. Should we create an issue about that? Ciao
          Hide
          Petr Skoda added a comment -

          +1 to remove smarty soon, does the qti_two actually work?

          Show
          Petr Skoda added a comment - +1 to remove smarty soon, does the qti_two actually work?
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Plz, consider my previous comment about Smarty. TIA!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks! Plz, consider my previous comment about Smarty. TIA!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Passing, not uses found, but the commented ones. Information sent to http://docs.moodle.org/dev/Moodle_2.2_release_notes#Libraries_deleted_from_distribution

          Please, don't forget the Smarty thingy.

          Show
          Eloy Lafuente (stronk7) added a comment - Passing, not uses found, but the commented ones. Information sent to http://docs.moodle.org/dev/Moodle_2.2_release_notes#Libraries_deleted_from_distribution Please, don't forget the Smarty thingy.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Done, your delicious hacks have been sent upstream, many thanks!

          Closing as fixed, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Done, your delicious hacks have been sent upstream, many thanks! Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: