WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 3 years ago

#57096 closed defect (bug) (invalid)

Post Featured Image - Link settings

Reported by: dhrumilk Owned by:
Priority: normal Milestone:
Component: Editor Version: 6.1.1
Severity: blocker Keywords: has-patch
Cc: Focuses: administration, coding-standards

Description

I have found the bug on the Post Featured Image block. i have use that block on site and set the image on features image block and also add LINK REL and test it but this link is not working on block, link redirect on same post URL, not on set URL.

Screen Shot 1:​https://www.awesomescreenshot.com/image/34351166?key=fd7b17cff868460301b077536890fbee

Screen Shot 2: ​https://www.awesomescreenshot.com/image/34350981?key=f9b55d9078ee5b9659a6aba7c5a0ba1e

Attachments (4)

image-1.png​ (208.6 KB ) - added by dhrumilk 4 years ago.
Screenshot 1
image-2.png​ (230.9 KB ) - added by dhrumilk 4 years ago.
Screenshot 2
57096.diff​ (711 bytes ) - added by dhrumilk 4 years ago.
57096.1.diff​ (710 bytes ) - added by dhrumilk 4 years ago.
Updated diff.

Download all attachments as: .zip

Change History (13)

This ticket was mentioned in ​PR #3613 on ​WordPress/wordpress-develop by ​dhrumilkumbhani.
4 years ago #1

  • Keywords has-patch added; needs-patch removed

I have update the code for set the LINK REL value on the Post Featured Image block.

Trac ticket:

#2 @dhrumilk
4 years ago

  • Keywords needs-testing has-patch removed

@dhrumilk
4 years ago

Screenshot 1

@dhrumilk
4 years ago

Screenshot 2

#3 @dhrumilk
4 years ago

  • Keywords close added
  • Resolution → invalid
  • Status new → closed

I have checked but now it's working perfectly.

#4 @desrosj
4 years ago

  • Component General → Editor
  • Keywords close removed
  • Milestone Awaiting Review

#5 @dhrumilk
4 years ago

  • Focuses accessibility removed
  • Resolution invalid
  • Status closed → reopened
  • Version 6.1 → 6.1.1

This issue is still there.

@dhrumilk
4 years ago

#6 @dhrumilk
4 years ago

  • Keywords has-patch added

#7 @jrf
4 years ago

I haven't checked the issue or the actual correctness of the proposed solution, but would like to note that in the current patch esc_attr() is the wrong escaping function. As this should be a URL, esc_url() should be used.

#8 @dhrumilk
4 years ago

@jrf thank you for the correct me.

@dhrumilk
4 years ago

Updated diff.

#9 @dhrumilk
3 years ago

  • Resolution → invalid
  • Status reopened → closed
Note: See TracTickets for help on using tickets.