Attempting to share a url with a query string.

Easy Social is ignoring the ?string part of my url.

CommentFileSizeAuthor
#3 easy_social-query_string_url-2500687-3.patch540 bytesrenatog
#3 Result.gif1.5 MBrenatog
#2 2.png15.13 KBrenatog
#2 1.png158.09 KBrenatog
#1 easy_social-query_string_url-2500687-1.patch841 bytesMediaFormat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MediaFormat’s picture

MediaFormat CreditAttribution: MediaFormat commented
Title: url query string ignored » Include query string in share url
Category: Bug report » Feature request
Status: Active » Needs review
FileSize
easy_social-query_string_url-2500687-1.patch841 bytes

Here is a tentative patch

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs review » Needs work
Issue tags: +ciandt-contrib
FileSize
1.png158.09 KB
2.png15.13 KB

Hi @mediaformat, how are you?

Your patch is ok, but show this error:

Because need pass 'query' parameter in url().

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Issue summary: View changes
Status: Needs work » Needs review
FileSize
Result.gif1.5 MB
easy_social-query_string_url-2500687-3.patch540 bytes

New patch with fix it's in attachment.

Result:

Thank you very much.

Regards.

  • rhormens committed 10f8eca on 7.x-2.x authored by RenatoG 
    Issue #2500687 by RenatoG, MediaFormat: Include query string in share...
rhormens’s picture

rhormens CreditAttribution: rhormens at CI&T commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.