Port Fasttoggle module to Drupal 7

Subscribe

Bump ;)

I can pay you a large cheese pizza. It's one less pizza I get to have, but it would be worth it.

Subscribe.

Subscribing...

I'm not a coder but ran the module through the coder module

Maybe someone can take it from here.

+++++++++++++++++++++++++++++
fasttoggle/fasttoggle.module
Summary
minor: 1, normal: 10, critical: 3, ignored: 0
Details
+2: [minor] Commits to the Git repository do not require the CVS $Id$ keyword in each file.
+9: [critical] Module .info files must now specify all loadable code files explicitly.
+58: [critical] %_comment wildcard has been renamed to %comment
+162: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+166: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+265: [critical] Parameters to hook_form_alter have changed.
+271: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+279: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+290: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+298: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+304: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+311: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+331: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+421: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE

+++++++++++++++++++++++++++++
fasttoggle/fasttoggle.toggle.inc
Summary
minor: 1, normal: 4, critical: 0, ignored: 0
Details
+-1: [normal] @file block missing
+2: [minor] Commits to the Git repository do not require the CVS $Id$ keyword in each file.
+13: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+78: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE
+130: [normal] Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE

+++++++++++++++++++++++++++++
fasttoggle/fasttoggle.install
Summary
minor: 1, normal: 0, critical: 0, ignored: 0
Details
+2: [minor] Commits to the Git repository do not require the CVS $Id$ keyword in each file.

+++++++++++++++++++++++++++++
fasttoggle/fasttoggle.admin.inc
Summary
minor: 1, normal: 1, critical: 0, ignored: 0
Details
+-1: [normal] @file block missing
+2: [minor] Commits to the Git repository do not require the CVS $Id$ keyword in each file.

Not yet - I'm still almost exclusively using D6 and have been working on other projects, but will happily start a D7 branch and commit patches.

subscribe

Will do. Just have to find the time :)

Upgrade your site to D7 and you'll find the time :) Sucks without it!

Started on this today. Code changes are done, up to debugging. Got distracted by other things, but I thought I'd give you some encouragement :)

HOOORAY! Beers on me.

Great news indeed!

Hell ya!

Just an update, so you know progress is happening:

I have the non-ajax side of things working and have started to learn the Ajax changes in D7 so I can do that side. I hope to find the time to do it tomorrow night, AEST.

7.x-1.0-beta1 is now in git. If I've done things right, a tarball should appear soon. If not, I'll figure it out :)

...and fixed ;)

This issue was for an initial D7 port. Now that we have a 7.x branch available, I believe that follow-up issues should be filed separately against that branch.

Thanx for your work Nigel!

PS: how about a 7.x-dev branch where future work will happen? Thanx for considering.

When I have something to put in, yes - I'll make a dev branch.

...that's done too ;)