CommentFileSizeAuthor
#2 ct-2086479-2.patch7.02 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

disasm’s picture

Status: Active » Closed (duplicate)
tim.plunkett’s picture

Issue summary: View changes
Status: Closed (duplicate) » Needs review
FileSize
7.02 KB

I really want confirm_form() dead (it's one of the last places using $_GET!)

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Even i kind of hate to force other people to reroll patches for the sake of own patches.

+++ b/core/modules/content_translation/lib/Drupal/content_translation/Form/ContentTranslationDeleteForm.php
@@ -0,0 +1,94 @@
+    if (\Drupal::moduleHandler()->moduleExists('path')) {

This is fine given that this is just temporary anyway.

tim.plunkett’s picture

Yeah I was halfway done injecting it when I realized it was a @todo and would be easier to fix later if left as-is.

tim.plunkett’s picture

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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