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

      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

        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: