Details

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

      Description

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

        Gliffy Diagrams

          Activity

          Hide
          skodak Petr Skoda 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
          skodak Petr Skoda 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
          mjollnir Penny Leach added a comment -

          eh?

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

          Show
          mjollnir Penny Leach added a comment - eh? I was talking about the portfolio caller class....
          Hide
          skodak Petr Skoda added a comment -

          eh? sorry :-D

          Show
          skodak Petr Skoda added a comment - eh? sorry :-D

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                24/Nov/10