Hi, this looks awesome. I'm currently using the chart api module on Drupal 5 and was hoping you might backport this module to make it Drupal 5 compatible. Unfortunately Drupal 6 isn't an option for my site as the required modules aren't released yet.

CommentFileSizeAuthor
#10 charts-d5.patch5.57 KBowen barton

Comments

brmassa’s picture

brmassa commented
Status: Active » Fixed

Tancredi,

i have to be honest, i really dont have the time to implement and maintain it. Im in the opposite situation: i must use D6 (coz my project will be launched in a couple of months). I would be glad to see and help some doing this.

regards,

massa

PS: im marking it as FIXED, but if you or somebody else want to continue, please, feel free to reopen it!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

TUc’s picture

TUc commented
Status: Closed (fixed) » Active

Hi,

It doesn't seem like an awful lot of work to backport this module, isn't it?
If you don't mind, I would like to give it a try.

regards,
TUc

brmassa’s picture

brmassa commented

Jan,

remember that this module is now part of Google Summer of Code and we are going to expand the module substantially ONLY for D6.

regards,

massa

ipwa’s picture

ipwa commented

I support Jan, even with how it works right now, it'd be awesome to see this module in D5, specially for the reports. So Jan, please try to port it, I know a lot more people that are really interested in this, and when I get some more free time, I would be able to help out porting. Thanks so much Bruno for this wonderfull module, it rocks! Some things don't work quite right yet (I realize this is an alpha release) like Open Flash Chart, I get a blank screen, but when I saw the Drupal reports in charts my jaw dropped! Obrigado cara :)

Nicolas

toemaz’s picture

toemaz commented

Interested in the D5.x port as well!

brmassa’s picture

brmassa commented
Status: Active » Fixed

Guys,

im sorry but im marking it as fixed again because im now implementing several new features for Charts that will be basically impossible to be backported.

Again, if someone do the port AND maintain the code, i will be glad to commit.

regards,

massa

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

mroswell’s picture

mroswell commented
Status: Closed (fixed) » Closed (won't fix)
owen barton’s picture

owen barton commented
StatusFileSize
new charts-d5.patch5.57 KB

Here is a basic patch to make charts work in Drupal 5. As you can see the delta is very small. I am not certain that I caught everything, but certainly the admin form and examples now work.

It's probably easier to maintain this as a patch here, rather than an official CVS branch, since it will be less work to keep up to date than having to commit all new code to 2 branches.