The preprocess function template_preprocess_lightbox2_insert_image() stands in the lightbox2.insert.inc file, but this is not specified in the lightbox2_theme() function.

The consequence is that, if the lightbox2.insert.inc file is not already included on theme registry build operation, the expected preprocess function is simply not added in the list of preprocess functions.

CommentFileSizeAuthor
#1 lightbox2-hook_theme-lightbox2_insert_image-1275910-1.patch462 bytesb-prod

Comments

b-prod’s picture

b-prod commented
Status: Active » Needs review
StatusFileSize
new lightbox2-hook_theme-lightbox2_insert_image-1275910-1.patch462 bytes

The patch below fixes this.

feng-shui’s picture

feng-shui commented

Patch works beautifully, without it when the insert button is cliched the href and src attributes for the link and image are empty.

Thanks!

b-prod’s picture

b-prod commented
Status: Needs review » Reviewed & tested by the community

According to Feng-Shui comment (#2), I tag this issue as reviewed.

pifagor’s picture

pifagor
Location 🇺🇦 Rivne
commented
Issue summary: View changes

Look good

alex_optim’s picture

alex_optim
Location Tallinn
commented

Good for me.

  • voleger committed c14a8c1 on 6.x-1.x authored by B-Prod 
    Issue #1275910 by B-Prod, voleger: File declaration missing in...
voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Reviewed & tested by the community » Fixed

Fixed. Thanks.

Status: Fixed » Closed (fixed)

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