I just got Bluff working with a temporary js fix. Can I have commit access to push the branch?

CommentFileSizeAuthor
#21 charts_graphs-d7port-1096702-20.patch132.42 KBbatje
#19 chart_graphs-d7port-1096702-19.patch130.43 KBmgifford
#5 charts_graphs-d7_port-1.patch16.51 KBthekevinday
charts_graphs-7.x-2.x-dev.tgz1.12 MBR.J. Steinert
charts_graphs-7.x-2.x-screenshot.png56.64 KBR.J. Steinert

Comments

R.J. Steinert’s picture

R.J. Steinert commented

*nudge nudge* :)

rsevero’s picture

rsevero commented
Assigned: Unassigned » rsevero
Priority: Critical » Normal
Status: Active » Needs work

Could you please post a patch with your changes?

R.J. Steinert’s picture

R.J. Steinert commented

I just rebased all the recent 6.x-2.x changes into my 7.x-2.x branch and it's a big mess. It might be less work to start over again than merge all of this together. This could be a lot easier if the api modules were separate projects. I think it makes sense for API integrations to have their own path because because, for example, I'm interested in moving the Flot integration and pCharts integration forward but none of the current integrations in the charts_graphs module. It's definitely "handy" for all of those integration modules to be included but I would prefer that the charts_graphs module move faster and not be slowed down by making sure a handful of integrations that are rarely used (by some) are updated in time.

SeanBannister’s picture

SeanBannister commented

sub

thekevinday’s picture

thekevinday commented
Status: Needs work » Needs review
StatusFileSize
new charts_graphs-d7_port-1.patch16.51 KB

I recreated a clean patch based off of rjstatic's changes.
This patch is against the latest git head 6.x-2.x.
Hopefully it should only contain d6 to d7 changes and nothing else.
The other fixes may be useful, but for transitioning it would be best to wait.

This needs review as I may have missed some changes.

batje’s picture

batje commented

subscribe

ishmael-sanchez’s picture

ishmael-sanchez commented

Trying the patch file I get a fatal error:

require_once(\sites\all\modules\charts_graphs/charts_graphs_canvas.class.inc) [function.require-once]: failed to open stream: Invalid argument in charts_graphs_init() (line 12 of \sites\all\modules\charts_graphs\charts_graphs.module).

I changed the paths but the module requires the SWF tools module but there is not D7 version. Suggestions?

ludo.r’s picture

ludo.r
Location Brussels
commented

Any news on D7 port?

R.J. Steinert’s picture

R.J. Steinert commented
Title: Here's 7.x-2.x-dev, can I have commit access? » Port charts_graphs to Drupal 7

There is some interest brewing in this over at http://drupal.org/node/828458 and http://groups.drupal.org/node/193923

ludo.r’s picture

ludo.r
Location Brussels
commented

I haven't checked lately, but I think D7 still miss good (views) chart module(s) while D6 do have some quite good modules.

I hope some contributors will be able to fullfil this need.

batje’s picture

batje commented

The flot module has recently gotten new maintainers too http://drupal.org/project/flot There is a first dev release out.

R.J. Steinert’s picture

R.J. Steinert commented

Nice, sounds like the Flot module for D7 has support for timestamps which is something I had to hack in charts_graphs and flot modules for D6.

If your X-axis field is a datetime field and if you have an exposed filter on this field with the between operator, you can zoom in on your graph with refreshing.

akshaynhegde’s picture

akshaynhegde commented
Priority: Normal » Critical

am too pissed off trying to use charts module with other than google charts. Charts & Graphs was working so well in d6 that I cant wait to get a d7 version

I tried using external libraries such as pchart, fusionchart, open-flash-chart etc and still haven't even able to create a single graph. even the example they have provided doesn't work properly. I need a D7 version of charts and graphs as all my work has got stuck.

So am requesting whoever is developing to release a D7 version as soon as possible.

WorldFallz’s picture

WorldFallz commented
Priority: Critical » Normal
Status: Needs review » Needs work

Please don't change issue priorities without knowing how.

The best way to push this forward is post an updated patch rather than demanding maintainers, who are already contributing their time for free, to work faster for your needs.

akshaynhegde’s picture

akshaynhegde commented

@WorldFallz , am really sorry i didn't exactly knew what those priorities meant. So thanks for pointing it out.
I as I said am completely stuck here. So I think you understand my desparation.

I hope the D7 version comes out soon.

batje’s picture

batje commented

You could try out the flot module. It works for D7 and it makes charts & graphs http://drupal.org/project/flot

akshaynhegde’s picture

akshaynhegde commented

I already tried that..!! Here is the problem.... http://drupal.org/node/1456256

Ii even tried using external librararies such as fusionchart,pchart,openflashchart etc but nothing is working properly..!

problem with the fusionchart... http://drupal.stackexchange.com/questions/23516/fusioncharts-with-drupal-7

problem with pchart... http://drupal.stackexchange.com/questions/23452/unusual-output-while-using-pchart-in-drupal-7

So at this moment am completely stuck..!! If you have got solution to any of these problems, please let me know.

cmputrgk’s picture

cmputrgk commented

Any word on the d7 version?

mgifford’s picture

mgifford
he/him
Primary language English
commented
StatusFileSize
new chart_graphs-d7port-1096702-19.patch130.43 KB

Sorry @thekevinday I didn't review your patch. I started with git for this project and then just ran it through Coder Upgrade & then Coder. I just wanted to do something quick...

I was looking at #1372558: Drupal 7 release? and had that all set up anyways.

So, hopefully this helps move it along. I fixed the problem defined in #7 too. It just wasn't using http://api.drupal.org/api/drupal/includes!common.inc/function/drupal_get...

batje’s picture

batje commented

Fixed some issues with paths to the include files.

Also updating the views_charts issue.

batje’s picture

batje commented
StatusFileSize
new charts_graphs-d7port-1096702-20.patch132.42 KB

with patch

batje’s picture

batje commented

I created a sandbox for the Drupal 7 upgrade. This version works with the Bluff library and with a new plugin we wrote for the flot library.

There is 3 things that I would like to propose and discuss with others:

  1. The module now contains submodules that implement a specific provider. For any maintainer/stakeholder it will be hard to support all of them. As with the flot implementation we just build, I suggest we split off the submodules and make them projects of their own. If there are volunteers for these subprojects, speak up!
  2. The whole module probably needs a big overhaul. It seems using ctools plugins is a good idea. What I think really lacks is a way for plugins to extend the settings form and secondly a way to set default systemwide settings (like, you only want to change your graphs colorscheme once in your site, not on every display of every view)
  3. We got the module this far, if the current maintainers dont have the time anymore, I would not mind helping them by moving my code into a new Drupal7 branch inside this project.
slithy’s picture

slithy commented

subscribe

batje’s picture

batje commented

@slithy there is a green button in the top of the right column of the original post (up ^^) that you can click to subscribe to issues

batje’s picture

batje commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Needs work » Closed (fixed)

Thanks all for your work. We have used your patches, and added some of ours to make a first -dev release.

If there are people who would like to take over the flash & google plugins, we would be happy to hand them over.

For a new and beautiful graphing option, try our Flot integration.

Happy Graphing everyone!