Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: DEV backlog
    • Fix Version/s: 2.1
    • Component/s: Quiz
    • Labels:
      None

      Gliffy Diagrams

        Attachments

          Issue Links

            Activity

            Hide
            nebgor Aparup Banerjee added a comment -

            i'm trying to create a proper 1.9 quiz backup test data set.

            Tim, i'm sure i won't be doing the best of job generating this data with all the possible options, could you provide a reasonable backup to test against ?

            Show
            nebgor Aparup Banerjee added a comment - i'm trying to create a proper 1.9 quiz backup test data set. Tim, i'm sure i won't be doing the best of job generating this data with all the possible options, could you provide a reasonable backup to test against ?
            Hide
            mudrd8mz David Mudrák added a comment -

            Tim is on holiday. It's all up to you soldier! Go go! Please note the quiz itself should not be that hard. Just do not forget to annotate used questions in inforef.xml. I am working on the question bank conversion.

            Show
            mudrd8mz David Mudrák added a comment - Tim is on holiday. It's all up to you soldier! Go go! Please note the quiz itself should not be that hard. Just do not forget to annotate used questions in inforef.xml. I am working on the question bank conversion.
            Hide
            nebgor Aparup Banerjee added a comment -

            David, i didn't create inforef that you mentioend in devchat yet.
            but can you have a review of this for now?

            Show
            nebgor Aparup Banerjee added a comment - David, i didn't create inforef that you mentioend in devchat yet. but can you have a review of this for now?
            Hide
            mudrd8mz David Mudrák added a comment -

            In process_quiz_question_instance(), why not to use write_xml() instead of the foreach loop? Same applies to process_quiz_feedback(). The write_xml() is intended for complete structures including end tag. We use foreach loop only when we can't close the wrapping tag yet. The rest is looking good to me (from a formal point of view without studying the upgrade logic).

            Show
            mudrd8mz David Mudrák added a comment - In process_quiz_question_instance(), why not to use write_xml() instead of the foreach loop? Same applies to process_quiz_feedback(). The write_xml() is intended for complete structures including end tag. We use foreach loop only when we can't close the wrapping tag yet. The rest is looking good to me (from a formal point of view without studying the upgrade logic).
            Hide
            nebgor Aparup Banerjee added a comment -

            thanks for the quick review David , yea i realised that too after en_tag() right after loop lol, was rushing to test it hehe...

            i'm assuming numeric precision data changes are taken care of while being inserted into the db , that should be ok.. unless these converters are used for other purposes.

            changing that stupid loop now..

            Show
            nebgor Aparup Banerjee added a comment - thanks for the quick review David , yea i realised that too after en_tag() right after loop lol, was rushing to test it hehe... i'm assuming numeric precision data changes are taken care of while being inserted into the db , that should be ok.. unless these converters are used for other purposes. changing that stupid loop now..
            Hide
            nebgor Aparup Banerjee added a comment -

            ok updated https://github.com/nebgor/moodle/compare/backup-convert...MDL-27447

            i've also received a big quiz zip from Ulrike Montgomery (thanks ) that i'll be testing with tomorrow. g'night!

            Show
            nebgor Aparup Banerjee added a comment - ok updated https://github.com/nebgor/moodle/compare/backup-convert...MDL-27447 i've also received a big quiz zip from Ulrike Montgomery (thanks ) that i'll be testing with tomorrow. g'night!
            Hide
            nebgor Aparup Banerjee added a comment -

            i've just rebased this branch onto david's current backup-convert branch

            Show
            nebgor Aparup Banerjee added a comment - i've just rebased this branch onto david's current backup-convert branch
            Hide
            nebgor Aparup Banerjee added a comment -

            David,
            this should be ready for your pre-integration branch.

            testing was fine with my small quiz backup. (attached)

            Show
            nebgor Aparup Banerjee added a comment - David, this should be ready for your pre-integration branch. testing was fine with my small quiz backup. (attached)
            Hide
            mudrd8mz David Mudrák added a comment -

            Waiting for the pull request at github

            Show
            mudrd8mz David Mudrák added a comment - Waiting for the pull request at github
            Show
            nebgor Aparup Banerjee added a comment - done @ https://github.com/mudrd8mz/moodle/pull/11
            Hide
            mudrd8mz David Mudrák added a comment -

            Done. Combined with the MDL-27626, this seems to be able to convert all standard question types.

            Show
            mudrd8mz David Mudrák added a comment - Done. Combined with the MDL-27626 , this seems to be able to convert all standard question types.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Jul/11