I just transferred a development site to a GoDaddy hosting environment and started seeing the error over a WSOD.
It works local (naturally) but production environment not so much.

I really appreciate all your hard work on this module and I wish I could detail the issue thoroughly but in short all I know is that I'm running PHP 5.2.17 so that shouldn't be the problem.

I'm not dumb, but the error logs for this domain aren't kicking in yet.
I expect it's a security glitch but as yet I have no idea.
I'll post more once I get a more indepth look whenever Godaddy starts playing ball.

Thanks again for your work on this module, it's got a lot of great potential and I hope to get it rocking and rolling.

CommentFileSizeAuthor
#1 define-1548158-1.patch313 bytesstevector

Comments

stevector’s picture

stevector
he/him
commented
Status: Active » Needs review
StatusFileSize
new define-1548158-1.patch313 bytes

Constants should be defined with define().

This bug also breaks drush.

barnettech’s picture

barnettech commented

I just looked up in the drupal coding standards manual what is right here:

http://drupal.org/coding-standards

In Drupal 8 and later, constants should be defined using the const PHP language keyword (instead of define()), because it is better for performance:

Also I ran drush and it is working fine in D7 with Droogle installed.

barnettech’s picture

barnettech commented

I do believe it's a PHP version problem. I'm running PHP 5.3.8 myself. I did fix this for the Drupal 6 version I seem to recall but figured for Drupal 7 I would go with the future and more current way of using constants which is also required for Drupal 8.

I just checked Acquia cloud is running PHP 5.2.4 so it may make sense to apply the patch above for now, until more folks move to > PHP 5.3.x

barnettech’s picture

barnettech commented
Status: Needs review » Fixed

Ok I applied the patch and created a new tag which you can download from drupal.org or via drush: 7.x-1.0-beta2

Let me know if you have any further problems.

barnettech’s picture

barnettech commented

btw thanks very much for the encouragement dooley, I will enjoy posting code more if I know it is useful to the community. And stevector thanks for the patch, saved me re-researching the error.

In the next month I'm due to add a bunch of new features to this module, it will come out for D6 first, support for folders is the main thing. Google Drive is out! This is what we were waiting for here at Babson :)

James
http://www.barnettech.com
http://www.activitiesincommon.com
and soon http://www.mypersonalvillage.com/

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

reworded