Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-14591 META: Develop new Portfolio API
  3. MDL-15884

refactor assignment implmentation

          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

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  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