As part of a separate patch, I was asked to clean up the URLs found on the admin page: https://www.drupal.org/node/1967520#comment-10278127

Rather than continue to include them in that patch, I've created a new one that only includes those updates.

Please review, thanks.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ron_s created an issue. See original summary.

ron_s’s picture

Status: Active » Needs review
FileSize
4.93 KB
Pol’s picture

Status: Needs review » Closed (outdated)
ron_s’s picture

Status: Closed (outdated) » Needs review

Why outdated? Looks to me as though these issues still apply.

ron_s’s picture

Status: Needs review » Closed (won't fix)

I give up on this patch. I'm told by @joelpittet I need to write it, then I'm told by @pol it is outdated.

A new version of the module is released last week, with no effort to address this. I've got better things to do than to be given conflicting messages for 3 years. Such a massive waste of time.

Pol’s picture

Hi,

Sorry it has been a bit complicated to sort all of this, I will fix this myself tomorrow.

Thanks.

ron_s’s picture

Sort all of this? It's a simple patch to use the standard Drupal approach to formatting URLs in strings. This isn't complicated.

Pol’s picture

Pol’s picture

Status: Closed (won't fix) » Fixed

Fixed.

joseph.olstad’s picture

For my drush make file
please ignore this patch otherwise.

Pol’s picture

I fixed it Joseph :-)

Status: Fixed » Closed (fixed)

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