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
    • Rank:
      19412

      Description

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

        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 Škoda added a comment -

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

        Show
        Petr Škoda 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: