Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#35405 closed defect (bug) (invalid)

Bug with lightbox-gallery

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

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 9 years ago.
screenshot for bugfix

Download all attachments as: .zip

Change History (3)

@m266
9 years ago

screenshot for bugfix

#1 follow-up: @Clorith
9 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

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
9 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.