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

POLICY: third party library deprecation warnings should be communicated direct to developers



    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: None
    • Component/s: Libraries, Policy
    • Labels:
    • Affected Branches:


      Recently on moodle.org/plugins, gallery-lightbox was functioning on moodle 2.5.2.
      upon upgrading to moodle 2.6, the gallery-lightbox started having issues (MDLSITE-2260).

      The issue i've realised here is that old YUI deprecated code was still functioning properly in 2.5.. It was functioning with deprecated YUI code while all warnings were NOT being shown to developer even in DEBUG DEVELOPER mode. This fails to use moodle developer warning system.

      In moodle 2.6, we had upgraded YUI to 3.13 and the deprecated function was finally completely removed. This caused breakage obviously and all that is there in release notes is 'upgraded to yui 3.13.'
      It was actually deprecated years ago: https://github.com/yui/yui3/commit/4599ac01a1b5d10be18c377d27f6a357b34220a7

      We need to :
      1) forward (bubble up with some filtering?) deprecation warnings of third party libs to developers (in developer debug mode).
      2) if there is no deprecation system in the third party lib to bubble up (hopefully rare), then we'll need to monitor what third party api we're changing and inform developers with a third party deprecation file/txt/script perhaps.

      ps:Andrew Nicols has kindly reminded yui guys of deprecation removals in their release notes. https://github.com/yui/yui3/issues/1415


          Issue Links



              • Assignee:
                dougiamas Martin Dougiamas
                nebgor Aparup Banerjee
                Component watchers:
                Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Adrian Greeve, Andrew Nicols, Eloy Lafuente (stronk7), Juan Leyva, Jun Pataleta, Sander Bangma
              • Votes:
                0 Vote for this issue
                5 Start watching this issue


                • Created: