Project

General

Profile

Feature #7408

Upgrade pChart to 2.x version

Added by Daniel Barreto about 6 years ago. Updated almost 6 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
Daniel Barreto
Category:
Tracking / Reporting
Target version:
1.10.0
Start date:
18/11/2014
Due date:
% Done:

100%

Estimated time:
Spent time:
21.34 h
Complexity:
Normal
SCRUM pts - complexity:
?

Description

The actual version of pChart library used in Chamilo is 1.27d, but actually the version 2.1.4 have been enhaced. Added more features and fixed bugs.
pChart 1.27 classes
  • pChart
  • pData
  • pCache
pChart 2.1.4 classes
  • pBarcode39
  • pBarcode128
  • pBubble
  • pCache
  • pData
  • pDraw
  • pImage
  • pIndicator
  • pPie
  • pRadar
  • pScatter
  • pSplit
  • pSpring
  • pStock
  • pSurface

A problem here is pChart class doesn't exist, and some methods used in Chamilo should be adapted to new pChart 2.x classes
Should consider changes for these commits:

https://github.com/chamilo/chamilo-lms/commit/468ded54a5a8b8456fe4c28c6dc8fb7011d55b0b
https://github.com/chamilo/chamilo-lms/commit/df256d657859b672571b2d3269d95cbd2b42ec93

Files

pchart.png (25 KB) pchart.png Alex Aragon, 03/02/2015 19:59
colores.png (16.3 KB) colores.png Alex Aragon, 05/02/2015 16:42

History

#1

Updated by Julio Montoya about 6 years ago

+1 for this.

#2

Updated by Daniel Barreto about 6 years ago

  • Tracker changed from Bug to Feature
#3

Updated by Daniel Barreto almost 6 years ago

  • Status changed from New to Assigned
  • Assignee set to Daniel Barreto
#4

Updated by Daniel Barreto almost 6 years ago

  • % Done changed from 0 to 10

Replace pchart old code from flatview.class.php file, display_graph_by_resource method
Update autoload
Work in:
https://github.com/danbarretodev/chamilo-lms/tree/CT7408

To do:
Update previous custom fixes to old pChart library
Check cache feature (archive folder)

#5

Updated by Daniel Barreto almost 6 years ago

  • % Done changed from 10 to 30

Upgrade the chart used in my progress

See:
https://github.com/danbarretodev/chamilo-lms/tree/CT7408

To do:
Upgrade Chart for surveys, dashboard and others
Define a global color template for pChart
see http://wiki.pchart.net/doc.color.schemes.html
Add a open source font (Verdana and Tahoma were used until now)

#6

Updated by Julio Montoya almost 6 years ago

+1

#7

Updated by Daniel Barreto almost 6 years ago

  • % Done changed from 30 to 70

Upgrade all pChart use

see:
https://github.com/danbarretodev/chamilo-lms/tree/CT7408

To do:
Review for code style and standards

#8

Updated by Daniel Barreto almost 6 years ago

  • Assignee changed from Daniel Barreto to Alex Aragon

Done:
Use now open sans font, downloaded to /main/css folder
Correct code style and standards
Replace reference to pchart library (old version) and delete it
Replace pChart2 library by https://packagist.org/packages/szymach/c-pchart
Update composer.json and composer.lock (only update c-pchart library), should generate new autoload files using

composer install

see https://getcomposer.org/doc/00-intro.md

To do:
Define a global color template for pChart
see http://wiki.pchart.net/doc.color.schemes.html

Send PR to 1.10.x:
https://github.com/chamilo/chamilo-lms/pull/463

#9

Updated by Daniel Barreto almost 6 years ago

Move opensans folder to /main/fonts.
Execute composer install to add pChart
Rebase commits onto 1.10.x

#10

Updated by Alex Aragon almost 6 years ago

Adjuntos algunos modelos de colore para usar.

#11

Updated by Yannick Warnier almost 6 years ago

El primero me gusta más.
Puedes enviar los códigos de color para que Daniel los use?

#12

Updated by Alex Aragon almost 6 years ago

y los colores son:

#13

Updated by Daniel Barreto almost 6 years ago

  • Assignee changed from Alex Aragon to Daniel Barreto

Add palette for graphs
Merge Chamilo 1.10.x branch
Add pChart classes use in flatview table
Test graphs

186, 206, 151
210, 148, 147
148, 170, 208
221, 133, 61
65, 153, 176
114, 88, 144
138, 166, 78
171, 70, 67
69, 115, 168

Update PR

https://github.com/chamilo/chamilo-lms/pull/463

#14

Updated by Yannick Warnier almost 6 years ago

  • Category set to Tracking / Reporting
  • Status changed from Assigned to Feature implemented
  • % Done changed from 70 to 100

This upgrade has been implemented and is working. Nice job!

Also available in: Atom PDF