Support from Acquia helps fund testing for Drupal Acquia logo

Comments

atchijov’s picture

atchijov & johnbickar working on this.

atchijov’s picture

atchijov’s picture

Status: Active » Needs review
atchijov’s picture

Assigned: Unassigned » atchijov
kim.pepper’s picture

Status: Needs review » Needs work
+++ b/core/modules/overlay/overlay.moduleundefined
@@ -151,7 +151,8 @@ function overlay_init() {
+    $query_render = \Drupal::request()->query->get('render');

Maybe rename this to $render?

+++ b/core/modules/overlay/overlay.moduleundefined
@@ -168,7 +169,7 @@ function overlay_init() {
+      \Drupal::request()->query->remove('render');

Let's extract a $query variable to be re-used.

Anonymous’s picture

Status: Needs work » Needs review
FileSize
792 bytes

Reroll the patch as the hook_init removed https://drupal.org/node/2013014.

Crell’s picture

Status: Needs review » Reviewed & tested by the community

I have no idea what black magic overlay is trying to do here, but at least this way it's doing it with the right API. :-)

Thanks!

Dries’s picture

Seems like this needs a re-roll.

Dries’s picture

Status: Reviewed & tested by the community » Needs work

The last submitted patch, drupal-use_symfony_request_fro_overlay_module-1999408-4.patch, failed testing.

Anonymous’s picture

Status: Needs work » Needs review
FileSize
649 bytes

Actually this code already committed.

$token = drupal_container()->get('request')->query->get('token');
if (!empty($_POST) || isset($token)) {

Now I Reroll the patch #6.So please Needs Review.

Crell’s picture

+++ b/core/modules/overlay/overlay.module
@@ -586,7 +586,8 @@
+  $post = Drupal::request()->query->count();

request()->query is the GET parameters. You want the POST parameters, which I believe are request()->request->count().

kim.pepper’s picture

FileSize
694 bytes
697 bytes

Fixes #12

Crell’s picture

Status: Needs review » Reviewed & tested by the community

That looks better. Thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 9d61e24 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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