Moodle
  1. Moodle
  2. MDL-14591 META: Develop new Portfolio API
  3. MDL-15884

refactor assignment implmentation

        Details

        • Type: Sub-task Sub-task
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.0
        • Fix Version/s: 2.0
        • Component/s: Portfolio API
        • Labels:
          None
        • Affected Branches:
          MOODLE_20_STABLE
        • Fixed Branches:
          MOODLE_20_STABLE
        • Rank:
          32915

          Description

          the caller class should defer to the assignment type classes rather than have hardcoded type logic. (similar to resource module)

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Petr Škoda added a comment -

            hmm, it needs a full rewrite, not only refactoring

            The grading must be moved into gradebook plugins, uploading has to be rewritten, the access control improved, etc. In fact the plugins idea did not work much here, it might be better to have custom grading plugins and one general assignment type. I was telling this for at least two years - I know Martin will not agree.

            Show
            Petr Škoda added a comment - hmm, it needs a full rewrite, not only refactoring The grading must be moved into gradebook plugins, uploading has to be rewritten, the access control improved, etc. In fact the plugins idea did not work much here, it might be better to have custom grading plugins and one general assignment type. I was telling this for at least two years - I know Martin will not agree.
            Hide
            Permalink
            Penny Leach added a comment -

            eh?

            I was talking about the portfolio caller class....

            Show
            Penny Leach added a comment - eh? I was talking about the portfolio caller class....
            Hide
            Permalink
            Petr Škoda added a comment -

            eh? sorry :-D

            Show
            Petr Škoda added a comment - eh? sorry :-D

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: