I ported the main code to Drupal 6 (not much to do, practically just the menu entry and the info), which version do you want me to diff against?

CommentFileSizeAuthor
#6 Chart 6.x-1.x-dev21.34 KBeinsicht

Comments

tjholowaychuk’s picture

Great! I have been meaning to port all of my modules just have not had time lately. Diff against 1.2 would be great

tjholowaychuk’s picture

Status: Active » Postponed
snufkin’s picture

sry i hardly have time. to save the effort i just upload the updated module tarball.

snufkin’s picture

doh the upload got lost. this is it

tjholowaychuk’s picture

Assigned: Unassigned » tjholowaychuk

Great thanks I will take a look soon ( low on time also :) )

einsicht’s picture

StatusFileSize
new21.34 KB

Hey tjholowaychuk,
I'm not an expert but I'd like to help. This is one of my first contributions to a module. Hope it's useful!

I took a little time to make Charts and System Charts work on D6. Didn't test much yet but it seems to work all right. I wanted to post it here ASAP so anyone that wants to help can use the code.

Changes:

  • in the .info-s to reflect new D6 requirements
  • .module-s: mainly hook_menu, hook_validate & chart_settings (i.e. for Menu system and FAPI).

I'll be working on extending this for a module I'm working on, but if you could use some help with this one I'm in!

tjholowaychuk’s picture

Thanks! I will diff/test/commit the changes ASAP.
Personally I have refrained from using this module until the Google summer of code students are finished with their work as one was working on a chart related combination of my chart module and another Drupal users.

einsicht’s picture

Oh, I didn't know that. Sounds like I should wait too. In your experience, do you have a rough estimate on when that Google Code work will come back to Drupal? Link or something to their plan/documentation? Is it gonna be an entirely new module or extend on this one?

Also, I tested it after uploading, and the "nodes" charts may need to be upgraded as well. Not sure if the API for nodes has changed. The Watchdog and Users works fine so far.

boombatower’s picture

Status: Postponed » Fixed

Seems like we have a Drupal 6 port.

Status: Fixed » Closed (fixed)

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