Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#35405 closed defect (bug) (invalid)

Bug with lightbox-gallery

Reported by: m266 Owned by:
Priority: normal Milestone:
Component: Gallery Version: 4.4
Severity: normal Keywords:
Cc: Focuses:

Description

Hi,
since WP 4.4 is this a bug: https://core.trac.wordpress.org/ticket/34826
And the bug is in WP 4.4.1 too.
Have you a solution for the next release?
Thanks.

Best regards

Attachments (1)

bugfix lightbox-gallery.png (42.2 KB ) - added by m266 11 years ago.
screenshot for bugfix

Download all attachments as: .zip

Change History (3)

@m266
11 years ago

screenshot for bugfix

#1 follow-up: @Clorith
11 years ago

  • Milestone Awaiting Review
  • Resolutioninvalid
  • Status newclosed

Hi, and welcome to Trac!

In this situation, we actually need the rel attribute to be a part of the variable, as you can see on line 130 we use the $rel variable directly:

$html = '<a href="' . esc_attr($url) . "\"$rel>$html</a>";

Since we output it here, we expect it to contain the rel attribute tag, as well as the content of the tag.

The adjustment mentioned in #34826 changes it so we send the values that would make the else part trigger.

If you are experiencing the double rel attributes still, I'd recommend trying to clear your local cache first, as it may just be something sticking around locally, but of course if you continue to experience it do follow up on this ticket and we'll be happy to revisit it with you.

#2 in reply to: ↑ 1 @m266
11 years ago

Hi Clorith,

thanks for your answer.
Lightbox Plus Colorbox worked not since WP 4.4 (cache is empty) without the small hack in media.php. I must then use another plugin probably unfortunately.

Best regards

Replying to Clorith:

Hi, and welcome to Trac!

In this situation, we actually need the rel attribute to be a part of the variable, as you can see on line 130 we use the $rel variable directly:

$html = '<a href="' . esc_attr($url) . "\"$rel>$html</a>";

Since we output it here, we expect it to contain the rel attribute tag, as well as the content of the tag.

The adjustment mentioned in #34826 changes it so we send the values that would make the else part trigger.

If you are experiencing the double rel attributes still, I'd recommend trying to clear your local cache first, as it may just be something sticking around locally, but of course if you continue to experience it do follow up on this ticket and we'll be happy to revisit it with you.

Note: See TracTickets for help on using tickets.