Moodle
  1. Moodle
  2. MDL-8395 Calculated question: adding or improving class (calculated, datasetdependent,numerical) functions
  3. MDL-8565

Calculated question:removing use of $SESSION as storage between steps of calculated questiion creation and edition process

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: None
    • Component/s: Questions
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE

      Description

      The actual three steps process of creating, editing or saving as new question use $SESSION to store data between the steps.
      The most crucial problem is that when creating a new question, the saving of question parameters is postponed to the third step until the user add a first dataitems. The proposition is to save the parameters at each steps so that if the user does not complete the process, what has been done is not lost.
      A mode detailed analysis is available at
      http://docs.moodle.org/en/Development:Calculated_question_bugs_and_new_features_proposal
      A solution has been tested and could be implement on 1.9 even 1.8 as it does not imply new database features.
      The necessary files will be available soon on
      http://moodleuqamchi.cvs.sourceforge.net/moodleuqamchi/moodle/
      and as attachment here

        Gliffy Diagrams

          Activity

          Hide
          Pierre Pichet added a comment -

          a .zip containing the 5 files and the diff as obtained using Tortoise cvs.
          These files has been added to a moodle head dailay download to test if everything work correctly.
          http://132.208.141.198/moodle_head/
          user: moodle pw: moodle Course Testing 1.8

          Some additional display of category shareable wild cards has been added to the edit_calculated_form and the datasetdefinitions_form.
          The text strings used will have to be corrected using the right getstring() when the code flow and options will be fixed.

          Show
          Pierre Pichet added a comment - a .zip containing the 5 files and the diff as obtained using Tortoise cvs. These files has been added to a moodle head dailay download to test if everything work correctly. http://132.208.141.198/moodle_head/ user: moodle pw: moodle Course Testing 1.8 Some additional display of category shareable wild cards has been added to the edit_calculated_form and the datasetdefinitions_form. The text strings used will have to be corrected using the right getstring() when the code flow and options will be fixed.
          Hide
          Pierre Pichet added a comment -

          The modifications has been CVS on HEAD and merged on 1.8_STABLE.
          Next step is to implement them on earlier version 1.7 and 1.6 as UQAM is on 1.6 for next year.

          Show
          Pierre Pichet added a comment - The modifications has been CVS on HEAD and merged on 1.8_STABLE. Next step is to implement them on earlier version 1.7 and 1.6 as UQAM is on 1.6 for next year.
          Hide
          Pierre Pichet added a comment -

          Done down to 1.6.
          there is some cleaning code to remove all unnecessary $SESSION reference ,so not closing yet as a remember...

          Show
          Pierre Pichet added a comment - Done down to 1.6. there is some cleaning code to remove all unnecessary $SESSION reference ,so not closing yet as a remember...

            People

            • Assignee:
              Pierre Pichet
              Reporter:
              Pierre Pichet
              Tester:
              Nobody
              Participants:
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: