What are the plans for migration of this module to Drupal 7? Any why I can help?

Comments

BarisW’s picture

Anyone? Would love to have this as well ;)

michelhack’s picture

+1 I could wonders with this

slucero’s picture

Component: Miscellaneous » Code
Status: Active » Needs review
StatusFileSize
new3.53 KB

Applying this patch on top of the 7.x-1.x branch as it sits right now produces a working copy of the module for Drupal 7. The patch mostly contains query function changes to work with Drupal 7, and the actual application of the classes to blocks has been modified to use the classes_array variable in the block_preprocess function.

slucero’s picture

Added another small adjustment to avoid a notice when no block reaction exists.

moondancerjen’s picture

When I applied this patch I am getting the following errors:

Hunk #2 FAILED at 80.
Hunk #3 FAILED at 109.
Hunk #4 FAILED at 128.
Hunk #5 FAILED at 170.

I downloaded the 7.x.-1 project from the link you supplied and successfully ran the patch in #3, but got these errors when trying to apply this patch.

Any ideas????

slucero’s picture

So you applied #3 and then tried to apply #4? The patch in #4 should be used by itself.

PCateNumbersUSA’s picture

Any change a Drupal 7 version can be release with this patch?

moondancerjen’s picture

slucero: I did apply both. I'll try to remove patch 3 and do patch#4 by itself.

In other news, I'm getting two errors on my site as a result of this module:
Notice: Undefined property: stdClass::$bid in context_block_class_reaction_block_class->execute() (line 70 of /pathtomydrupaldir/sites/all/modules/contrib/context_block_class/plugins/context_block_class_reaction_block_class.inc).
^same message shows up twice. Saw a related issue for the D6 version of this module, but the fix wasn't posted, so I'm not sure how to apply it to this version of the module. May have something to do with being on PHP 5.3???

Notice: Undefined variable: context_block_classes in include() (line 45 of /pathtomydrupaldir/sites/all/themes/mytheme/templates/block.tpl.php).
^this is from me adding the line to block.tpl.php Is this step necessary for the D7 version?

moondancerjen’s picture

Oh, and one more update! Because of these issues, my block classes aren't actually printing...

moondancerjen’s picture

Another update! Sorry for so many comments!

I tried again and downloaded the D7 source and did only patch #4 and that removed the error.

I still do not have the block classes printing, and I think that is because of this line in the context_block_class.inc file:

// @todo Add support for more than just the default theme.
    $theme = variable_get('theme_default', 'garland');

I'm definitely not using garland. So, how can I add support for my default theme?

SolomonGifford’s picture

Status: Needs review » Reviewed & tested by the community

We applied the patch in #4 and can confirm it works with D7.

ttkaminski’s picture

A second confirmation that the patch works.

chrisjlee’s picture

Which is the working patch #4?

gausarts’s picture

+1 RTBC