Quantcast

[Bug 17282] New: Ability to create charts for SQL reports

classic Classic list List threaded Threaded
41 messages Options
12
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] New: Ability to create charts for SQL reports

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

            Bug ID: 17282
           Summary: Ability to create charts for SQL reports
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P5 - low
         Component: Reports
          Assignee: [hidden email]
          Reporter: [hidden email]
        QA Contact: [hidden email]

Add a form under report's result that allow to configure and draw a chart (pie,
bar, line and combination).

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Alex Arnaud <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|[hidden email]          |[hidden email]

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #1 from Alex Arnaud <[hidden email]> ---
Created attachment 55412
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55412&action=edit
Bug 17282 - Ability to create charts for SQL reports

Add a form under report's result that allow to configure and draw a
chart (pie, bar, line and combination).

Pie: Usefull only for a two-column report's result

bar: Horizontal: Can be horizontal or vertical (check/uncheck
horizontal checkbox),
     Group: allows to group columns (stacked bar chart),
     Line: show some columns as line in a bar chart (combination)

line: line chart :)

This patch adds 2 new js libraries: d3js and c3js:
  - c3.min.css
  - c3.min.js
  - d3.min.js

Test plan:
 - Apply this patch,
 - make reports,
 - draw charts,
 - that's all

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Owen Leonard <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |Needs Signoff

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Héctor Eduardo Castro Avalos <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55412|0                           |1
        is obsolete|                            |

--- Comment #2 from Héctor Eduardo Castro Avalos <[hidden email]> ---
Created attachment 55442
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55442&action=edit
[SIGNED-OFF]Bug 17282: Ability to create charts for SQL reports

Add a form under report's result that allow to configure and draw a
chart (pie, bar, line and combination).

Pie: Usefull only for a two-column report's result

bar: Horizontal: Can be horizontal or vertical (check/uncheck
horizontal checkbox),
     Group: allows to group columns (stacked bar chart),
     Line: show some columns as line in a bar chart (combination)

line: line chart :)

This patch adds 2 new js libraries: d3js and c3js:
  - c3.min.css
  - c3.min.js
  - d3.min.js

Test plan:
 - Apply this patch,
 - make reports,
 - draw charts,
 - that's all

Signed-off-by: Hector Castro <[hidden email]>
Works as advertised

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Héctor Eduardo Castro Avalos <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off
                 CC|                            |[hidden email]

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #3 from Héctor Eduardo Castro Avalos <[hidden email]> ---
Could be better if we can only use the dropdown menu to download the chart.
I propose this patch, if you think that is bad idea, please make it obsolete.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Héctor Eduardo Castro Avalos <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Needs Signoff

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #4 from Héctor Eduardo Castro Avalos <[hidden email]> ---
Created attachment 55444
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55444&action=edit
Bug 17282: (follow-up) Use drop-down menu for download chart instead

To Test
1) Execute a report
2) Click in drop-down menu "Download" shows only three options
3) Draw a chart
4) Click again in drop-down menu "Download" shows a new option "Download chart"

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Alex Arnaud <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55444|0                           |1
        is obsolete|                            |

--- Comment #5 from Alex Arnaud <[hidden email]> ---
Created attachment 55480
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55480&action=edit
Bug 17282: (follow-up) Use drop-down menu for download chart instead

To Test
1) Execute a report
2) Click in drop-down menu "Download" shows only three options
3) Draw a chart
4) Click again in drop-down menu "Download" shows a new option "Download chart"

Signed-off-by: Alex Arnaud <[hidden email]>

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Héctor Eduardo Castro Avalos <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |[hidden email]-c
                   |                            |ommunity.org

--- Comment #6 from Jonathan Druart <[hidden email]> ---
QA comments:
1/ Would it be possible to add a "loading please wait" when processing the
graph?
2/ Execute a report which raises an error, you will get a JS error:
3/ If I delete a column, the scrollbar goes to the top
4/ I can delete all the columns and click on "draw", nothing happens. It should
not bz possible to delete the first column I'd say.
5/ With the following query:
SELECT i.homebranch, h.branchcode, monthname(h.reservedate) AS month,
       year(h.reservedate) AS year, count(*)
FROM old_reserves h
LEFT JOIN items i USING (itemnumber)
WHERE h.branchcode != i.homebranch AND year(h.reservedate) = '2015'
GROUP BY month(h.reservedate), h.branchcode

With chart type: "Pie", x column: "month" and y: "count(*)", I get the
following JS error:
  TypeError: columns[index] is undefined

Marked as Failed QA.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #7 from Alex Arnaud <[hidden email]> ---
(In reply to Jonathan Druart from comment #6)

> 2/ Execute a report which raises an error, you will get a JS error:

Have you an exemple of report that raise error ?

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #8 from Jonathan Druart <[hidden email]> ---
(In reply to Alex Arnaud from comment #7)
> (In reply to Jonathan Druart from comment #6)
>
> > 2/ Execute a report which raises an error, you will get a JS error:
>
> Have you an exemple of report that raise error ?

SELECT itemtype, count(*) FROM items group by itemtype;
should do the trick

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #9 from Alex Arnaud <[hidden email]> ---
Created attachment 55582
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55582&action=edit
Bug 17282 - Fix QA comments:

- Don't scroll to the top when deleting a column,
  - prevents to delete all columns,
  - better handling of headers/results deleting

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Alex Arnaud <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Needs Signoff

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Alex Arnaud <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55582|0                           |1
        is obsolete|                            |

--- Comment #10 from Alex Arnaud <[hidden email]> ---
Created attachment 55586
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55586&action=edit
Bug 17282 - Fix QA comments:

- Don't scroll to the top when deleting a column,
  - prevents to delete all columns,
  - better handling of headers/results deleting

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #11 from Alex Arnaud <[hidden email]> ---
Created attachment 55588
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55588&action=edit
Bug 17282 - Ability to exclude the last line in report results

This patch add a checkbox (exclude last line). This is useful
when this last is the result of a sql rollup which is the sum
of each column and make the chart not consistent

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Josef Moravec <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55442|0                           |1
        is obsolete|                            |

--- Comment #12 from Josef Moravec <[hidden email]> ---
Created attachment 55604
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55604&action=edit
Bug 17282: Ability to create charts for SQL reports

Add a form under report's result that allow to configure and draw a
chart (pie, bar, line and combination).

Pie: Usefull only for a two-column report's result

bar: Horizontal: Can be horizontal or vertical (check/uncheck
horizontal checkbox),
     Group: allows to group columns (stacked bar chart),
     Line: show some columns as line in a bar chart (combination)

line: line chart :)

This patch adds 2 new js libraries: d3js and c3js:
  - c3.min.css
  - c3.min.js
  - d3.min.js

Test plan:
 - Apply this patch,
 - make reports,
 - draw charts,
 - that's all

Signed-off-by: Hector Castro <[hidden email]>
Works as advertised

Signed-off-by: Josef Moravec <[hidden email]>

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Josef Moravec <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55480|0                           |1
        is obsolete|                            |

--- Comment #13 from Josef Moravec <[hidden email]> ---
Created attachment 55605
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55605&action=edit
Bug 17282: (follow-up) Use drop-down menu for download chart instead

To Test
1) Execute a report
2) Click in drop-down menu "Download" shows only three options
3) Draw a chart
4) Click again in drop-down menu "Download" shows a new option "Download chart"

Signed-off-by: Alex Arnaud <[hidden email]>

Signed-off-by: Josef Moravec <[hidden email]>

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Josef Moravec <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55586|0                           |1
        is obsolete|                            |

--- Comment #14 from Josef Moravec <[hidden email]> ---
Created attachment 55606
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55606&action=edit
Bug 17282 - Fix QA comments:

- Don't scroll to the top when deleting a column,
  - prevents to delete all columns,
  - better handling of headers/results deleting

Signed-off-by: Josef Moravec <[hidden email]>

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Josef Moravec <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Signed Off

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Josef Moravec <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55588|0                           |1
        is obsolete|                            |

--- Comment #15 from Josef Moravec <[hidden email]> ---
Created attachment 55607
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55607&action=edit
Bug 17282 - Ability to exclude the last line in report results

This patch add a checkbox (exclude last line). This is useful
when this last is the result of a sql rollup which is the sum
of each column and make the chart not consistent

Signed-off-by: Josef Moravec <[hidden email]>

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #16 from Alex Arnaud <[hidden email]> ---
Currently, graphs are built using the data of the page (Because this the only
available variable in the template). For reports with paging, this means that
all data are not took in account. I'm wondering if we should not make that
possible. How ? Get all data with an AJAX call during making the graph ? It
seems to be a big work to AJAXify this part of the code.

Feedback are welcome

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #17 from Jonathan Druart <[hidden email]> ---
(In reply to Alex Arnaud from comment #16)
> Currently, graphs are built using the data of the page (Because this the
> only available variable in the template). For reports with paging, this
> means that all data are not took in account. I'm wondering if we should not
> make that possible. How ? Get all data with an AJAX call during making the
> graph ? It seems to be a big work to AJAXify this part of the code.
>
> Feedback are welcome

It seems reasonable, for a first version, to built the graph with the data
displayed on the page.
But it would be nice to tell the user how it works with a small info box.

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Katrin Fischer <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #18 from Katrin Fischer <[hidden email]> ---

> It seems reasonable, for a first version, to built the graph with the data
> displayed on the page.
> But it would be nice to tell the user how it works with a small info box.

+1

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

Jonathan Druart <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #19 from Jonathan Druart <[hidden email]> ---
(In reply to Jonathan Druart from comment #6)

Some of the comments from comment 6 have not been fixed!

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #20 from Jonathan Druart <[hidden email]> ---
Created attachment 55868
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55868&action=edit
Bug 17282: Use fieldset.rows

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #21 from Jonathan Druart <[hidden email]> ---
Created attachment 55869
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=55869&action=edit
Bug 17282: Remove use of onclick

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[Bug 17282] Ability to create charts for SQL reports

bugzilla-daemon
In reply to this post by bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17282

--- Comment #22 from Alex Arnaud <[hidden email]> ---
Created attachment 62529
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=62529&action=edit
Bug 17282 - Fix QA comments   - Add a loading message,   - prevent JS error
when report fails,   - fix (bootstrap 3 ?) displaying problems

--
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[hidden email]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
12
Loading...