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

Sitestats User Activity and lessons pages that are recursive

    XMLWordPrintable

Details

    • Verified
    • Verified
    • Hide

      This issue may not be easy to test:

      1. create 2 lessons page that have a circular dependency
      2. have some students generate some activity on those pages
      3. then use the Sitestats -> User Activity feature
      4. If User Activity shows the correct activity for students then this passed
      5. If you notice any messages in the log or the student activity is incorrect then it failed

       

      Show
      This issue may not be easy to test: create 2 lessons page that have a circular dependency have some students generate some activity on those pages then use the Sitestats -> User Activity feature If User Activity shows the correct activity for students then this passed If you notice any messages in the log or the student activity is incorrect then it failed  

    Description

      Traversing Lessons Hierarchies needs to take into account hierarchies that maybe recursive.

      For example if Lessons has a page which links to another page and then links back to the page it referenced.

      Particularly these methods look suspect but there maybe others:
      org.sakaiproject.sitestats.impl.event.detailed.refresolvers.LessonsReferenceResolver#getPageHierarchyReverse
      org.sakaiproject.sitestats.tool.transformers.LessonsResolvedRefTransformer#getPageDisplay

      These large strings have been noticed while analyzing the heap, see attachments.

      Gliffy Diagrams

        Zeplin

          Attachments

            Issue Links

              Activity

                People

                  plukasew Paul Lukasewych
                  ern Earle R Nietzel
                  Votes:
                  0 Vote for this issue
                  Watchers:
                  13 Start watching this issue

                  Dates

                    Created:
                    Updated:
                    Resolved:

                    Git Integration