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

PHP-OLE is GPL-incompatible

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.5.5, 2.6, 2.6.2, 2.7
    • Fix Version/s: 2.8
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      1) Check the excel files in the following areas and make sure that they download properly.

      • Bulk user download.
      • Gradebook.
      • Choice responses (report).
      • Feedback module (Analysis tab).
      • Scorm reports.
      • Survey.

      2) Also do a grep for files with OLE. Make sure that there are none. (git grep -i OLE_ |grep -iv role|grep -iv console|grep -iv whole|grep -v creole)

      Show
      1) Check the excel files in the following areas and make sure that they download properly. Bulk user download. Gradebook. Choice responses (report). Feedback module (Analysis tab). Scorm reports. Survey. 2) Also do a grep for files with OLE. Make sure that there are none. (git grep -i OLE_ |grep -iv role|grep -iv console|grep -iv whole|grep -v creole)
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-45395-master
    • Story Points:
      40
    • Sprint:
      BACKEND Sprint 13

      Description

      We removed it in MDL-42381..

      But as reported by Debian, we have another instance in lib/phpexcel/PHPExcel/Shared/OLE*

      https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=746594

      Note that in http://docs.moodle.org/dev/Moodle_libraries_credits#PEAR_OLE_Classes

      It's specified that an 'exception' was made for Moodle. That implies relicensing and is all bit hazy. If we don't need it we should get rid of it..

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Nov/14