Please update supercron.classes.inc to adhere to the Drupal coding standards, most notably the indenting. Thanks.

CommentFileSizeAuthor
#4 supercron.line-endings.patch93.47 KBpillarsdotnet

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 6.x-1.4-beta1 » 6.x-2.x-dev

It also seems that most files use Windows line endings instead of UNIX, please re-save everything in your text editor in UNIX format and re-commit, otherwise it'll be a mess trying to submit patches for most tasks. Thanks.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

FYI if you can fix the line endings problems (I don't think we can't provide a patch for it), I'll submit some patches for the other formatting issues :)

MisterSpeed’s picture

MisterSpeed commented

We're actually saving it in NetBeans under Ubuntu; I`ll double-check the settings there and push the files anew. Thanks for the heads up !

pillarsdotnet’s picture

pillarsdotnet commented
Status: Active » Needs review
StatusFileSize
new supercron.line-endings.patch93.47 KB

Here's a patch, but it ain't pretty.

Accomplished the change by running, at the command line:

perl -000 -pi -e 's,\r\n,\n,g' $(find * -type f | egrep -v '\.(swf|gif|png)')
pillarsdotnet’s picture

pillarsdotnet commented

Abandoned module?