user statistics sendjasonrequest error

Bug #1778451 reported by Cecilia Vela Gurovic on 2018-06-25
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Medium
Cecilia Vela Gurovic
17.04
Medium
Unassigned
17.10
Medium
Unassigned
18.04
Medium
Unassigned
Robert Lyon (robertl-9) wrote :

If I remember right this is a race condition problem as the datasets does get defined and graph does display correctly

Changed in mahara:
assignee: nobody → Cecilia Vela Gurovic (ceciliavg)
importance: Undecided → Low
status: New → Confirmed
Changed in mahara:
status: Confirmed → In Progress
importance: Low → Medium
Steven (stevens-q) wrote :

Environment tested: Master
Browser tested: Chrome

-------------------------------------
Manual Test Script
-------------------------------------

Preconditions:

1. Database must have a few users, institutions, content etc (to be able to see a report)

Test Script - Site admin runs Information overview report and checks to see graphs have a legend

1. Site admin logged in
2. Browse to Admin menu > Reports
3. User should see "Information overview | All institutions" title
4. Confirm the circular graphs are displayed with corresponding legends ✔
5. Confirm the line graph is displayed with corresponding legend ✔
6. Open the developer toolbar to view the colsole and refresh the page
7. Confirm there are no errors displayed ✔

[BUG] the graph legends are sometimes displayed correctly and upon a hard
refresh the graph legends may fall outside the containing box or display
over top of other page text. see attached screen shot. refreshing the page
after a hard reset will often correct the display error.

Steven (stevens-q) wrote :
Steven (stevens-q) wrote :

Bug Fixed - the graph legends are now displaying correctly after hard refresh and refresh of the Overview report page.

Catalyst QA Approved ✔

Reviewed: https://reviews.mahara.org/8981
Committed: https://git.mahara.org/mahara/mahara/commit/d09994c2b7df4a30f8336a5f55638767f9607f87
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit d09994c2b7df4a30f8336a5f55638767f9607f87
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Jun 29 13:01:33 2018 +1200

Bug 1778451: Show legend in charts

Use correct variable in template when creating the legend for charts.

In Chart.js legendTemplate uses var 'datasets'
or 'segments' depending on the type of charts
type Doughnut/PolarArea/Pie use segments
type Line/Radar/Bar use datasets

We have to check which one is defined in the Chart object before
modifying the legend template

Also delayed the updating of page by .5 secs so not to clash with
chart jsons stuff

behatnotneeded

Change-Id: Id3ef35a5037ac2c30d342279e0af7052584f7d16

Changed in mahara:
status: In Progress → Fix Committed
milestone: none → 18.10.0

Reviewed: https://reviews.mahara.org/8999
Committed: https://git.mahara.org/mahara/mahara/commit/c07156122d6dc32a2bb398635b1cd0b04521b101
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 18.04_STABLE

commit c07156122d6dc32a2bb398635b1cd0b04521b101
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Jun 29 13:01:33 2018 +1200

Bug 1778451: Show legend in charts

Use correct variable in template when creating the legend for charts.

In Chart.js legendTemplate uses var 'datasets'
or 'segments' depending on the type of charts
type Doughnut/PolarArea/Pie use segments
type Line/Radar/Bar use datasets

We have to check which one is defined in the Chart object before
modifying the legend template

Also delayed the updating of page by .5 secs so not to clash with
chart jsons stuff

behatnotneeded

Change-Id: Id3ef35a5037ac2c30d342279e0af7052584f7d16
(cherry picked from commit d09994c2b7df4a30f8336a5f55638767f9607f87)

Reviewed: https://reviews.mahara.org/9000
Committed: https://git.mahara.org/mahara/mahara/commit/7b9303eb875ed46b8f9d32c80aac032ef30d88b2
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 17.10_STABLE

commit 7b9303eb875ed46b8f9d32c80aac032ef30d88b2
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Jun 29 13:01:33 2018 +1200

Bug 1778451: Show legend in charts

Use correct variable in template when creating the legend for charts.

In Chart.js legendTemplate uses var 'datasets'
or 'segments' depending on the type of charts
type Doughnut/PolarArea/Pie use segments
type Line/Radar/Bar use datasets

We have to check which one is defined in the Chart object before
modifying the legend template

Also delayed the updating of page by .5 secs so not to clash with
chart jsons stuff

behatnotneeded

Change-Id: Id3ef35a5037ac2c30d342279e0af7052584f7d16
(cherry picked from commit d09994c2b7df4a30f8336a5f55638767f9607f87)
(cherry picked from commit c07156122d6dc32a2bb398635b1cd0b04521b101)

Reviewed: https://reviews.mahara.org/9001
Committed: https://git.mahara.org/mahara/mahara/commit/118984ac6d5d0bae3500fd1c739d5730c4e34d6a
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 17.04_STABLE

commit 118984ac6d5d0bae3500fd1c739d5730c4e34d6a
Author: Cecilia Vela Gurovic <email address hidden>
Date: Tue Jul 10 10:21:34 2018 +1200

Bug 1778451: Show legend in charts

Use correct variable in template when creating the legend for charts.

In Chart.js legendTemplate uses var 'datasets'
or 'segments' depending on the type of charts
 type Doughnut/PolarArea/Pie use segments
 type Line/Radar/Bar use datasets

We have to check which one is defined in the Chart object
before modifying the legend template

behatnotneeded

Change-Id: I3185616d5ffde2378c092c59c78c01503d01e6ae

Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/8981
Committed: https://git.mahara.org/mahara/mahara/commit/d09994c2b7df4a30f8336a5f55638767f9607f87
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit d09994c2b7df4a30f8336a5f55638767f9607f87
Author: Cecilia Vela Gurovic <email address hidden>
Date: Fri Jun 29 13:01:33 2018 +1200

Bug 1778451: Show legend in charts

Use correct variable in template when creating the legend for charts.

In Chart.js legendTemplate uses var 'datasets'
or 'segments' depending on the type of charts
type Doughnut/PolarArea/Pie use segments
type Line/Radar/Bar use datasets

We have to check which one is defined in the Chart object before
modifying the legend template

Also delayed the updating of page by .5 secs so not to clash with
chart jsons stuff

behatnotneeded

Change-Id: Id3ef35a5037ac2c30d342279e0af7052584f7d16

Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers