Any chance this module will be updated for Drupal 4.6? Right now, I get the follwoing error if I try to run clipper on 4.6:

warning: Invalid argument supplied for foreach() in /home/gov/WWW/drupal/modules/clipper/clipper.module on line 68.

warning: in_array(): Wrong datatype for second argument in /home/gov/WWW/drupal/modules/clipper/clipper.module on line 72.

warning: Invalid argument supplied for foreach() in /home/gov/WWW/drupal/modules/clipper/clipper.module on line 68.

warning: in_array(): Wrong datatype for second argument in /home/gov/WWW/drupal/modules/clipper/clipper.module on line 72.

CommentFileSizeAuthor
#5 clipper.module.patch249 bytesmgifford
#3 clipper.patch1008 bytesbeate_r

Comments

jasonwhat’s picture

jasonwhat commented

Yes, is this moduel dead?

Bèr Kessels’s picture

Bèr Kessels commented

no its not. Just don't expect an immediate update. If you require an immediate update, then iether send in a patch, or hire someone to make and send one.

beate_r’s picture

beate_r commented
StatusFileSize
new clipper.patch1008 bytes

will the enclosed patch solve the problem?

Bèr Kessels’s picture

Bèr Kessels commented

aybe, but it is hard for me to read your patch.
Please try to stick to the drupal patching convenstions. I will see if this does the trick.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Priority: Critical » Normal
Status: Active » Needs review
StatusFileSize
new clipper.module.patch249 bytes

I think this patch focuses more closely on the error messages on line 68 & 72 that are popping up if php warning messages are enabled.

I've tested it against the original code. The previous patch had other stuff in it.

Bèr Kessels’s picture

Bèr Kessels commented
Title: Error with Drupal 4.6 » Error with clipper array
Status: Needs review » Needs work

1) there is no clipper for 4.6
2) please use unified patches (http://drupal.org/patch)
3) this patch failed on the latest clipper.

jasonwhat’s picture

jasonwhat commented
Title: Error with clipper array » No plans for 4.6?

Are there still no plans for this to be available for 4.6?

Bèr Kessels’s picture

Bèr Kessels commented
Title: No plans for 4.6? » Error with clipper array

again: Unless there is a patch coming in for 4.6, the answer is NO.

And please, do not change the title!

Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs work » Fixed

Can no longer reproduce in the latest version.

Anonymous’s picture

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