Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Expected behavior
I expect that if I enter a legitimate Pinterest board URL, media_entity_pinterest's field formatter will render it.
Actual behavior
If I enter a legitimate Pinterest board URL that contains the % symbol, the board is not rendered.
Steps to reproduce
- Log into a Pinterest account.
- Create a new board with the plus symbol + in the name, and save.
- Copy the resulting generated URL, which should contain
%2Bin it. - Attempt to use this URL in a media_entity_pinterest field.
Results: board does not render.
Screenshots
Fix details
The regex patterns that validate Pinterest URLs do not allow for % in the board portion of the URL.
Board sections currently do allow for this and do not need to be modified.
Through testing, I have found that % is not allowed in user names, and so will not modify that pattern at this time.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3070119-allow_urlencoded_board_slugs-2.patch | 1.95 KB | timcosgrove |
| |||
| board_with_plus.png | 349.78 KB | timcosgrove | |
Comments
Comment #2
timcosgrove CreditAttribution: timcosgrove at Pinterest commentedThis adds support for
%and test data for the 4 types of supported URLs.Comment #3
timcosgrove CreditAttribution: timcosgrove at Pinterest commentedComment #5
gg4 CreditAttribution: gg4 commented