Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-43101

Clean up session state usage in Announcements

    XMLWordPrintable

    Details

    • Type: Branch
    • Status: OPEN
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 21.0 [Tentative]
    • Component/s: Announcements
    • Labels:
      None
    • Test Plan:
      Hide

      Please add a Test Plan here.

      Show
      Please add a Test Plan here.

      Description

      There are plenty of cleanup opportunities that have been idle for years. However, one of the problems that persists, and in some ways stymies cleanup, is the use of the session as an effective global variable bag throughout the large velocity Action classes.

      I propose some general refactoring with the goal of reducing and eliminating this spurious use of session state in view. There will certainly be some items that are not strictly related, but seem ripe to handle as broken windows. The main priority will be decomposing the giant classes into smaller ones with explicit dependencies and test coverage, such that changing the current, highly coupled implementation without breaking it becomes tractable through careful extraction. This should ease other, standing cleanup needs for the long term as well.

        Gliffy Diagrams

          Zeplin

            Attachments

              Activity

                People

                Assignee:
                maintenanceteam Core Team
                Reporter:
                noahbotimer Noah Botimer
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                  Dates

                  Created:
                  Updated:

                    Git Integration