Moodle
  1. Moodle
  2. MDL-21801

Non-functioning Powerpoint import option from the Lesson module

    Details

    • Testing Instructions:
      Hide
      1. Create new lesson for a course
        Make sure there's no option to import power point file and there's not error display.
      Show
      Create new lesson for a course Make sure there's no option to import power point file and there's not error display.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      2572

      Description

      Please remove Import Powerpoint option from the Lesson module because it does not work properly, because PowerPoint is a proprietary format and this non-working feature causes a number of false expectations and frustrations in moodle users.
      See http://moodle.org/mod/forum/discuss.php?d=145317

        Issue Links

          Activity

          Hide
          Chris Collman added a comment -

          There are related MDLs to this which are listed in: http://docs.moodle.org/en/Import_PowerPoint#See_also

          Today I agree with Joseph, which is a change from my thinking In 2006 and 2007 There has been a continued stream of posts about importppt and now a lack of importpptX in the Lesson Forum. From an increasingly growing pptx user base, it appears to be broken, While it should be removed form the standard, it should ALSO be placed in contributed code for the old ppt users.

          Being able to create pages from other sources that have content is important.. This saves data entry and provides the basis for transforming a forward/back series into an adaptive series of content pages. An additional reason for importing is that these other sources tend to have an easier interface to basically sort, edit and change the content , that the Lesson activity interface.

          Show
          Chris Collman added a comment - There are related MDLs to this which are listed in: http://docs.moodle.org/en/Import_PowerPoint#See_also Today I agree with Joseph, which is a change from my thinking In 2006 and 2007 There has been a continued stream of posts about importppt and now a lack of importpptX in the Lesson Forum. From an increasingly growing pptx user base, it appears to be broken, While it should be removed form the standard, it should ALSO be placed in contributed code for the old ppt users. Being able to create pages from other sources that have content is important.. This saves data entry and provides the basis for transforming a forward/back series into an adaptive series of content pages. An additional reason for importing is that these other sources tend to have an easier interface to basically sort, edit and change the content , that the Lesson activity interface.
          Hide
          Stuart R Mealor added a comment -

          Hmm, well I think it was my comment that suggested we should completely re-think the Moodle Docs page on PowerPoint import.
          I agree with Joseph taking the idea further and that the code/feature should be removed, although Moodle 2.0 might be too soon, perhaps for Moodle 2.1 (which is activity focused).
          It causes constant confusion and frustration for many teachers.
          Of course, it would be preferable to have a .ppt/.odf import that works

          Show
          Stuart R Mealor added a comment - Hmm, well I think it was my comment that suggested we should completely re-think the Moodle Docs page on PowerPoint import. I agree with Joseph taking the idea further and that the code/feature should be removed, although Moodle 2.0 might be too soon, perhaps for Moodle 2.1 (which is activity focused). It causes constant confusion and frustration for many teachers. Of course, it would be preferable to have a .ppt/.odf import that works
          Hide
          Martin Dougiamas added a comment -

          Reassigning this to moodle.com for the time being, since Mark Nielsen is not maintaining Lesson any more. Please comment/vote/patch any crucial bugs that affect you for 2.0 to help us prioritise fixes for the upcoming release.

          Show
          Martin Dougiamas added a comment - Reassigning this to moodle.com for the time being, since Mark Nielsen is not maintaining Lesson any more. Please comment/vote/patch any crucial bugs that affect you for 2.0 to help us prioritise fixes for the upcoming release.
          Hide
          Joseph Rézeau added a comment -

          BUMP ! Any hope of seeing the Powerpoint import feature removed from the Moodle Lesson module once for all in current versions of Moodle?

          Show
          Joseph Rézeau added a comment - BUMP ! Any hope of seeing the Powerpoint import feature removed from the Moodle Lesson module once for all in current versions of Moodle?
          Hide
          Alexandra added a comment -

          Until this feature will be resolved, it is better to be removed. There are several workarounds but not very good. Repair it and then let us play with it. So... it's a vote up!

          Show
          Alexandra added a comment - Until this feature will be resolved, it is better to be removed. There are several workarounds but not very good. Repair it and then let us play with it. So... it's a vote up!
          Hide
          Helen Foster added a comment -

          Just adding a link to the forum discussion mentioning this issue http://moodle.org/mod/forum/discuss.php?d=193631

          Show
          Helen Foster added a comment - Just adding a link to the forum discussion mentioning this issue http://moodle.org/mod/forum/discuss.php?d=193631
          Hide
          Alexandr Tatarenko added a comment - - edited

          Seems that I've solved the bug. Patch remove Import Powerpoint option from the Lesson module.

          Show
          Alexandr Tatarenko added a comment - - edited Seems that I've solved the bug. Patch remove Import Powerpoint option from the Lesson module.
          Hide
          Mary Cooch added a comment -

          Just wanted to bump this again as another user was confused by it on the Moodle forums today

          Show
          Mary Cooch added a comment - Just wanted to bump this again as another user was confused by it on the Moodle forums today
          Hide
          Joseph Rézeau added a comment -

          @Mary,
          I fail to understand why the Powerpoint Import option has not yet been removed once for all from the Lesson activity. This requests has gathered 13 votes, it keeps making users confused, it has NEVER worked properly, it should never have been part of the Lesson module to start with.
          I do not know what more we can do...

          Show
          Joseph Rézeau added a comment - @Mary, I fail to understand why the Powerpoint Import option has not yet been removed once for all from the Lesson activity. This requests has gathered 13 votes, it keeps making users confused, it has NEVER worked properly, it should never have been part of the Lesson module to start with. I do not know what more we can do...
          Hide
          Joseph Rézeau added a comment -

          BUMP! Is anyone listening? The Powerpoint import option must be removed from the Lesson activity, as it makes users falsely believe they can import Powerpoint files and then they post to the Lesson forum for help.

          Show
          Joseph Rézeau added a comment - BUMP! Is anyone listening? The Powerpoint import option must be removed from the Lesson activity, as it makes users falsely believe they can import Powerpoint files and then they post to the Lesson forum for help.
          Hide
          Mary Cooch added a comment -

          I am listening but I have no powers Bump bump bump!!

          Show
          Mary Cooch added a comment - I am listening but I have no powers Bump bump bump!!
          Hide
          Rossiani Wijaya added a comment -

          Hi everyone,

          Sorry for the long delay.

          I will start working on this.

          Show
          Rossiani Wijaya added a comment - Hi everyone, Sorry for the long delay. I will start working on this.
          Hide
          Ankit Agarwal added a comment -

          Hi Rosie,
          Reviewed the following:
          [Y] Syntax
          [NA] Output
          [Y] Whitespace
          [NA] Language
          [NA] Databases
          [Y] Testing
          [NA] Security
          [N] Documentation (This needs documentation and needs to be mentioned in upgrade.txt(may be?))
          [Y] Git
          [Y] Sanity check

          Reviewing this in master. The patch works as expected.
          Thanks

          Show
          Ankit Agarwal added a comment - Hi Rosie, Reviewed the following: [Y] Syntax [NA] Output [Y] Whitespace [NA] Language [NA] Databases [Y] Testing [NA] Security [N] Documentation (This needs documentation and needs to be mentioned in upgrade.txt(may be?)) [Y] Git [Y] Sanity check Reviewing this in master. The patch works as expected. Thanks
          Hide
          Dan Poltawski added a comment -

          Just commented to Rosie privately, my +1 for this to apply to the stable branches too as its broken. But should have ui_change tag

          (i'd say no to upgrade.txt, since thats really for developers, and I think the notice is for the users)

          Show
          Dan Poltawski added a comment - Just commented to Rosie privately, my +1 for this to apply to the stable branches too as its broken. But should have ui_change tag (i'd say no to upgrade.txt, since thats really for developers, and I think the notice is for the users)
          Hide
          Rossiani Wijaya added a comment -

          Thanks for commenting Dan.

          I created patch for 2.2 and 2.3 and added 'ui_change' and 'docs_required' labels.

          Submitting this for peer review.

          Show
          Rossiani Wijaya added a comment - Thanks for commenting Dan. I created patch for 2.2 and 2.3 and added 'ui_change' and 'docs_required' labels. Submitting this for peer review.
          Hide
          Mary Cooch added a comment -

          Great This will be a real pleasure to document!

          Show
          Mary Cooch added a comment - Great This will be a real pleasure to document!
          Hide
          Dan Poltawski added a comment -

          For more ammo to completely remove this from the stable branches:
          "marycooch: I'm just thinking of all the people who ask on forums why their imported powerpoint doesn't work - they will no longer be sent down a misleading path.."

          Mary, please comment about these things more! It makes the integrating decision much easier!

          Show
          Dan Poltawski added a comment - For more ammo to completely remove this from the stable branches: "marycooch: I'm just thinking of all the people who ask on forums why their imported powerpoint doesn't work - they will no longer be sent down a misleading path.." Mary, please comment about these things more! It makes the integrating decision much easier!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Sorry, but is it correct to delete the strings in stable branches...? Shouldn't we keep them there forever, no matter the functionality is out? (I know it's broken and so on, so anybody using older stable won't get anything useful... but... stable strings shouldn't be deleted ever IMO).

          I'm reopening this for a final iteration...ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Sorry, but is it correct to delete the strings in stable branches...? Shouldn't we keep them there forever, no matter the functionality is out? (I know it's broken and so on, so anybody using older stable won't get anything useful... but... stable strings shouldn't be deleted ever IMO). I'm reopening this for a final iteration...ciao
          Hide
          CiBoT added a comment -

          Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

          Show
          CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
          Hide
          Rossiani Wijaya added a comment -

          Yes, you are right. String shouldn't be removed from stable.

          I updated the patch by putting the string back to stables.

          Re-submit for integration review.

          Show
          Rossiani Wijaya added a comment - Yes, you are right. String shouldn't be removed from stable. I updated the patch by putting the string back to stables. Re-submit for integration review.
          Hide
          Dan Poltawski added a comment -

          Thanks Rosie, i've integrated this now.

          (I also bumped the version number in order to reflect the differing 'state' of the lesson module without this functionality)

          Show
          Dan Poltawski added a comment - Thanks Rosie, i've integrated this now. (I also bumped the version number in order to reflect the differing 'state' of the lesson module without this functionality)
          Hide
          Michael de Raadt added a comment -

          Test result: Success!

          The option is gone. Farewell.

          Show
          Michael de Raadt added a comment - Test result: Success! The option is gone. Farewell.
          Hide
          Aparup Banerjee added a comment -

          Your issue has dug up some gold.
          It works great i've been told.
          Go forth, be brave, be bold.

          yay! "All your thoughts are belong to everyone."

          Thanks and ciao!

          Show
          Aparup Banerjee added a comment - Your issue has dug up some gold. It works great i've been told. Go forth, be brave, be bold. yay! "All your thoughts are belong to everyone." Thanks and ciao!
          Hide
          Mary Cooch added a comment -

          Removing docs_required as it's now documented here http://docs.moodle.org/23/en/Building_Lesson#Import_a_powerpoint and in 2.2 docs. Joyous day

          Show
          Mary Cooch added a comment - Removing docs_required as it's now documented here http://docs.moodle.org/23/en/Building_Lesson#Import_a_powerpoint and in 2.2 docs. Joyous day
          Hide
          Joseph Rézeau added a comment -

          As the original reporter of this bug I'd like to thank all who voted for, watched and contributed to the removal of this obnoxious feature from the Lesson activity. And thanks also to Mary for amending the docs accordingly.
          Bye-bye, Powerpoint import option
          Joseph

          Show
          Joseph Rézeau added a comment - As the original reporter of this bug I'd like to thank all who voted for, watched and contributed to the removal of this obnoxious feature from the Lesson activity. And thanks also to Mary for amending the docs accordingly. Bye-bye, Powerpoint import option Joseph

            People

            • Votes:
              24 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: