$path = isset($current_path) ? $current_path : '<front>';
      global $base_url;
      $path_obj = Url::fromUri($base_url . '/' . $path,
        array(
          'absolute' => TRUE,
        )

in \Drupal\sharethis\SharethisManager::blockContents could use base://$path instead.

CommentFileSizeAuthor
#2 2769897.patch959 bytesnaveenvalecha
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner created an issue. See original summary.

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer and at Acquia for Acquia commented
Status: Active » Needs review
FileSize
2769897.patch959 bytes

Thanks for your posting issue @dawehner
It would be great if you post more issues/feedback if you find after your review

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner at Chapter Three commented

This works for me :)

naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer and at Acquia for Acquia commented
Status: Needs review » Fixed

Thanks!

  • naveenvalecha committed 094e09d on 8.x-2.x 
    Issue #2769897 by naveenvalecha, dawehner: Use base:// instead of $...
naveenvalecha’s picture

naveenvalecha
He/Him
Primary language Hindi
CreditAttribution: naveenvalecha as a volunteer and at Acquia for Acquia commented
Parent issue: » #2764639: Sharethis - 8.x-2.0-beta4

Status: Fixed » Closed (fixed)

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