After upgrading module from 7.x-1.3 to 7.x-1.5 it stops working. Links can be added in Block UI but block title doesn't not convert to url.

After looking inside changes I found that hook_preprocess_block() was changed to hook_process_block() in #1774180: calling preprocess function in a module should be http://cgit.drupalcode.org/block_titlelink/commit/?id=eef7748ec61ce2277c.... This is not a best solition because it doesn't work for all themes.

I'm using AdaptiveTheme and new version of module doesn't work for this theme.

CommentFileSizeAuthor
#2 block_titlelink-replace-2554461-2.patch518 bytesPlazik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Plazik created an issue. See original summary.

Plazik’s picture

Status: Active » Needs review
FileSize
518 bytes

I've attached patch for this.

adammalone’s picture

Status: Needs review » Postponed (maintainer needs more info)

Could you address the comment in #1774180: calling preprocess function in a module should be about Drupal coding standards and help me understand why we should use hook_preprocess_block instead of hook_process_block?

joshuautley’s picture

#2 Patch worked for us. We too use the AT theme. Thank you!

webservant316’s picture

Patch worked for me also! Thanks!

stinky’s picture

Yay! #2 Patch worked for me. Thanks.

  • typhonius committed 23240e1 on 7.x-1.x authored by Plazik
    Issue #2554461 by Plazik: Replace hook_process_block() with...
adammalone’s picture

Thanks for the input and testing, I've committed this to 7.x-1.x.

adammalone’s picture

Status: Postponed (maintainer needs more info) » Fixed

Status: Fixed » Closed (fixed)

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