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

Add calendar read events to sitestats

    Details

    • Type: Bug
    • Status: Verified
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 12.3
    • Fix Version/s: 19.0
    • Component/s: Calendar, Sitestats
    • Labels:
      None
    • Test Plan:
      Hide
      • Add Calendar and SiteStats to a site
      • Add a calendar event
      • Visit the SiteStats tool
      • Go to SiteStats Preferences and confirm Calendar > View calendar appears in the list of tools and events. If it is not currently selected, select it.
      • View the calendar event
      • Return to SiteStats
      • On the Overview tab, confirm that your view calendar events appear in the graphs
      • On the Reports tab, create a new report that shows events. Confirm that view calendar events appear here as well
      Show
      Add Calendar and SiteStats to a site Add a calendar event Visit the SiteStats tool Go to SiteStats Preferences and confirm Calendar > View calendar appears in the list of tools and events. If it is not currently selected, select it. View the calendar event Return to SiteStats On the Overview tab, confirm that your view calendar events appear in the graphs On the Reports tab, create a new report that shows events. Confirm that view calendar events appear here as well

      Description

      Calendar read events are commented out in the SiteStats event definition file. This patch uncomments them so they can be counted.

      It also slightly changes the calendar.read event reference to remove the id of the most recently viewed calendar item, which was being added to subsequent events where it was not appropriate. For example, if you visit the calendar with the weekly default view, the calendar.read reference looks like "<site_id>/week". If you view a calendar event, the calendar.read reference looks like this: "<site_id>/description/<cal_event_id>". If you then go back to the weekly view, the reference looks like "<site_id>/week/<cal_event_id>", which makes little sense as the id references a particular event, not a particular week.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  plukasew Paul Lukasewych
                  Reporter:
                  plukasew Paul Lukasewych
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code