Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using a colon in an asset title truncates the title in the page builder token listYour bug title goes here! #13676

Open
1 task done
solipsist opened this issue Apr 20, 2024 · 2 comments · May be fixed by #13678
Open
1 task done
Labels
assets Anything related to assets bug Issues or PR's relating to bugs mautic-5 Relates to Mautic 5.x T1 Low difficulty to fix (issue) or test (PR)
Milestone

Comments

@solipsist
Copy link

solipsist commented Apr 20, 2024

Mautic Version

5.0.x series

Way of installing

I downloaded a release from https://www.mautic.org/mautic-releases

PHP version

8.1.27

What browsers are you seeing the problem on?

No response

What happened?

Using a colon in the title of an asset makes the token only show the string before that colon, truncating the rest.

https://global.discourse-cdn.com/business7/uploads/mautic/original/2X/2/2c552adafc846340594d3d86b4db6fd1e108782d.png

https://global.discourse-cdn.com/business7/uploads/mautic/original/2X/5/567a8ec224cbadc2b7540c6ebbb17398311c387f.png

How can we reproduce this issue?

Step 1: Create an asset with a title containing a colon.
Step 2: Confirm that the token for that asset is truncated.

Relevant log output

No response

Code of Conduct

  • I confirm that I have read and agree to follow this project's Code of Conduct




Care about this issue? Want to get it resolved sooner? If you are a member of Mautic, you can add some funds to the Bounties Project so that the person who completes this task can claim those funds once it is merged by a member of the core team! Read the docs here.

@solipsist solipsist added bug Issues or PR's relating to bugs needs-triage For new issues/PRs that need to be triaged labels Apr 20, 2024
@dsp76
Copy link

dsp76 commented Apr 20, 2024

Confirmed... is recreatable.

@dsp76 dsp76 added T1 Low difficulty to fix (issue) or test (PR) mautic-5 Relates to Mautic 5.x and removed needs-triage For new issues/PRs that need to be triaged labels Apr 20, 2024
@RCheesley RCheesley added assets Anything related to assets good first issue labels Apr 29, 2024
@RCheesley
Copy link
Sponsor Member

Please test the PR #13678 @solipsist @dsp76 as it should resolve this issue for you. Thanks!

Docs on how to test can be found at https://mau.tc/tester - you can do it in the browser.

@RCheesley RCheesley added this to the 5.1.1 milestone Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets Anything related to assets bug Issues or PR's relating to bugs mautic-5 Relates to Mautic 5.x T1 Low difficulty to fix (issue) or test (PR)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants