Should not translate URL path ?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, remove_t.patch, failed testing.

droplet’s picture

Status: Needs work » Needs review

remove_t.patch queued for re-testing.

oriol_e9g’s picture

Status: Needs review » Reviewed & tested by the community

Good catch!

Berdir’s picture

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll

Needs a reroll...

git ac https://drupal.org/files/remove_t_0.patch
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1100  100  1100    0     0   1422      0 --:--:-- --:--:-- --:--:--  1864
error: patch failed: core/modules/dblog/lib/Drupal/dblog/Tests/DbLogTest.php:350
error: core/modules/dblog/lib/Drupal/dblog/Tests/DbLogTest.php: patch does not apply
shnark’s picture

Assigned: Unassigned » shnark

I'm going to reroll this, using these instructions https://drupal.org/patch/reroll

Unitoch’s picture

Assigned: shnark » Unitoch

I'm going to attempt a reroll.

Unitoch’s picture

Assigned: Unitoch » Unassigned
Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1.1 KB

Sorry about the cross-posting, Ella!

Here's a rerolled patch.

droplet’s picture

Status: Needs review » Needs work
+++ b/core/modules/dblog/lib/Drupal/dblog/Tests/DbLogTest.phpundefined
@@ -350,13 +350,13 @@ private function doNode($type) {
+    $this->assertText('node/@nid', array('@nid' => $node->id()), 'DBLog event was recorded: [page not found]');

Incorrect

droplet’s picture

Issue tags: +Novice, +Needs reroll

tagging

oriol_e9g’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1.08 KB
droplet’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed b4dd669 and pushed to 8.x. Thanks!

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