[wp-trac] [WordPress Trac] #57957: Media title falsely has kebab case when inserted via REST API (e.g. in Block Editor)
WordPress Trac
noreply at wordpress.org
Mon Mar 20 17:01:25 UTC 2023
#57957: Media title falsely has kebab case when inserted via REST API (e.g. in
Block Editor)
--------------------------+---------------------
Reporter: abitofmind | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: 6.2.1
Component: REST API | Version: 6.1.1
Severity: major | Resolution:
Keywords: has-patch | Focuses:
--------------------------+---------------------
Old description:
> == Reproduction I: "Upload New Media" creates media title "My Great
> Photo"
> 1) Media > Add New
> 2) Drag-n-drop "My Great Photo.jpg"
> 3) Inspecting this in "Edit Media" shows a media title "My Great Photo".
>
> == Reproduction II: Insertion via Block Editor (Gutenberg) creates media
> title "My-Super-Photo"
> 1) Edit an existing page or create a new page.
> 2) Set the cursor into an empty block.
> 3) Drag-n-drop "My Super Photo.jpg" into this block. This automatically
> creates a media page behind the scenes.
> 4) Save the page.
> 5) Inspecting the media title via "Edit Media"
> - **Actual:** The media title is "My-Super-Photo".
> - **Expected:** The media title should be "My Super Photo".
> - The media title should be the filename portion (without the suffix)
> in its purest possible form (UTF-8) and only the slug and filename should
> then be transformed according settings regarding
> ascii/whitespace/transliteration/escaping/punycode/etc.
> - Achieving the same goal via different means (Upload New Media vs.
> drag-n-drop in editor) should result in the same outcome. Otherwise this
> is an inconsistency.
>
> == Analysis & Fix
> @adamsilverstein already analyzed and fixed this bug in the Gutenberg
> issue tracker and asked to re-file here in WordPress Core for review &
> adding to the codebase.
>
> === Adam's Analysis
>
> > This is actually something that happens on the core side where the
> logic to set the title is currently different in the REST API vs. the
> regular media uploader.
>
> === Adam's Fix which should be reviewed
>
> > I proposed a fix in [https://github.com/WordPress/wordpress-
> develop/pull/3981 WordPress/wordpress-develop #3981] that works in my
> testing. Would be good to have some additional feedback, maybe from the
> REST API team.
>
> === Environment
>
> - First [https://github.com/WordPress/gutenberg/issues/34149 reported on
> 2021-08-19] in WordPress v5.8 ,
> [https://github.com/WordPress/gutenberg/issues/34149#issuecomment-1416260545
> confirmed/patched in 2023-02 ] on a WordPress v6.1.1 instance.
>
> === Priority
>
> - If the fix is deemed solid and the review process goes quickly without
> bumps: Is there a chance that the fix can make it into the 6.2 release?
> If not, no worry, am happy whenever this will get eventually fixed.
New description:
== Reproduction I: "Upload New Media" creates media title "My Great Photo"
1) Media > Add New
2) Drag-n-drop "My Great Photo.jpg"
3) Inspecting this in "Edit Media" shows a media title "My Great Photo".
== Reproduction II: Insertion via Block Editor (Gutenberg) creates media
title "My-Super-Photo"
1) Edit an existing page or create a new page.
2) Set the cursor into an empty block.
3) Drag-n-drop "My Super Photo.jpg" into this block. This automatically
creates a media page behind the scenes.
4) Save the page.
5) Inspecting the media title via "Edit Media"
- **Actual:** The media title is "My-Super-Photo".
- **Expected:** The media title should be "My Super Photo".
- The media title should be the filename portion (without the suffix) in
its purest possible form (UTF-8) and only the slug and filename should
then be transformed according settings regarding
ascii/whitespace/transliteration/escaping/punycode/etc.
- Achieving the same goal via different means (Upload New Media vs.
drag-n-drop in editor) should result in the same outcome. Otherwise this
is an inconsistency.
== Analysis & Fix
@adamsilverstein already analyzed and fixed this bug in the Gutenberg
issue tracker and asked to re-file here in WordPress Core for review &
adding to the codebase.
=== Adam's Analysis
> This is actually something that happens on the core side where the logic
to set the title is currently different in the REST API vs. the regular
media uploader.
=== Adam's Fix which should be reviewed
> I proposed a fix in [https://github.com/WordPress/wordpress-
develop/pull/3981 WordPress/wordpress-develop #3981] that works in my
testing. Would be good to have some additional feedback, maybe from the
REST API team.
=== Environment
- First [https://github.com/WordPress/gutenberg/issues/34149 reported on
2021-08-19] in WordPress v5.8 ,
[https://github.com/WordPress/gutenberg/issues/34149#issuecomment-1416260545
confirmed/patched in 2023-02 ] on a WordPress v6.1.1 instance.
=== Priority
- If the fix is deemed solid and the review process goes quickly without
bumps: Is there a chance that the fix can make it into the 6.2 release? If
not, no worry, am happy whenever this will get eventually fixed.
== References
* Gutenberg: [https://github.com/WordPress/gutenberg/issues/34149
Discussion and investigation details] where it was determined to be a Core
issue.
--
Comment (by hellofromTonya):
Adding a link to the original bug report from Gutenberg.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57957#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list