Download & Extend

Is it planned to make a release of the Checkall module for Drupal 7?

Project:Checkall
Version:6.x-2.x-dev
Component:Code
Category:support request
Priority:normal
Assigned:Unassigned
Status:reviewed & tested by the community
Issue tags:D7 porting, d7 ports, port to d7

Issue Summary

Are there any plans to make a version for Drupal 7?

Comments

#1

Yup. I'm curious about that too!

#2

Sure. Though, I'm not sure when I'll have the time.

If you're interested, I would be happy to grant maintainership access to anyone of you, both.

#3

Thanks. I wish I had the time to take it on.

If I need to upgrade it myself I'll certainly post a patch. And thanks if and when you do get the time. Not easy to do.

#4

into core

#5

Hello all

I have tried to port this to D7, this are the steps I have done:

checkall.module
Line 116
Before
$element['#attributes']['class'] = 'form-checkall '. $class;
After
$element['#attributes']['class'] = array('form-checkall '. $class);
Line 119
Before
$element['#attributes']['class'] .= ' form-checkall '. $class;
After
  $element['#attributes']['class'] .= array(' form-checkall '. $class);

Line 129
Before
$element[$key]['#attributes']['class'] = $class;
After
$element[$key]['#attributes']['class'] = array($class);

Line 132
Before
  $element[$key]['#attributes']['class'] .= ' '. $class;
After
  $element[$key]['#attributes']['class'] .= array(' '. $class);

checkall.js
Addded
(function ($) {
at the start of the code and
})(jQuery);
at the end of the code

Modified behaviors function:
Before

Drupal.behaviors.checkAll = function(context) {

  $('.form-checkall:not(.checkall-processed)', context).each(function() { Drupal.checkAll.attach(this); });
};

After
Drupal.behaviors.checkAll = {

attach: function(context,settings) {

  $('.form-checkall:not(.checkall-processed)', context).each(function() { Drupal.checkAll.attach (this,settings);});
}

};

References about this change:
http://drupal.org/node/756722 (see BEHAVIORS)

And now the last change, all references to
Drupal.settings.checkall have been change to settings. only
Example:
Line 27 was:

if (Drupal.settings.checkall && Drupal.settings.checkall.groups) {

Now is:
if (settings.checkall && settings.checkall.groups) {

Obviusly, you will have to change the checkall.info to work in D7 too :-P

I know this is not a patch and pershaps has a lot of bugs, but Im not a developer.
If someone with more knowledge can check this, I'm sure a lot of us will be very grateful

Hope this helps you all who needs this module working in D7 and thanks a lot to Markus for develop it to D6

#6

Title:Is it planned to make a release for Drupal 7?» Port Checkall module to drupal 7.

...less vague title + proper tags.

#7

Title:Port Checkall module to drupal 7.» Is it planned to make a release for Drupal 7?

The title is not vague. This is a support request that is asking to the maintainer if he has planned to make a Drupal 7 version of the module. To my question, the current maintainer has already replied; for this reason, this report could be closed as fixed.

I had two modules that used the Checkall module, but I removed the code using it because of the absence of a Drupal 7 version.
When I will have time, I will check if I can covert the module for Drupal 7; in that case, I will probably ask to become co-maintainer, once I tested the code.

#8

Title:Is it planned to make a release for Drupal 7?» Is it planned to make a release of the Checkall module for Drupal 7?

...sorry Alberto, by "vague" I was referring to the issue's title not mentioning which project it is for. Have you tried how a bunch of "D7 port?" issues for various projects look in your dashboard when they actually don't mention what project they are for?

#9

Hello

I have donde a patch for the version 6.x-2.4

Try it and let me know if it works, please

For install the patch:
-Download the 6.x-2.4 Version
-Unzip it and open a console in the unzipped dir
-Copy the patch to the unzipped dir and execute

patch -p1 < checkallportD7.patch

Good luck

AttachmentSize
checkallportD7.patch 4.97 KB

#10

@klonos Well, in few cases the name of the module is added to the issue title; one of those cases is when the issue is about a module being abandoned, or about taking over an existing module. Do you see the word "Drupal" in the title of all the issues opened for Drupal core?

#11

@kiamlaluno: go to your Drupal.org dashboard and make sure the "Your posts" block is enabled. Now, subscribe or post something in a few of the hundreds issues about porting modules/themes to D7. Take a look at your dashboard and imagine all of these posts having no specific project name in their title. It would look something like this. Try to figure out to which project they refer to simply by looking at the issue titles.

...having fun with those issues that do not include the project name?

Now, the same thing in the issue queue is entirely different since it includes project names in a separate column of their own.

#12

Status:active» reviewed & tested by the community

I can confirm that breos' patch in comment #9 works.