replaced $_POST with \Drupal::request()->request->get method [#2842312]

To get $_POST variables in Drupal 8 we need to use \Drupal::request()->request->get method.

Comment	File	Size	Author
#2	patch-2842312.patch	3.62 KB	chaitanya17

Comments

Comment #1

10 January 2017 at 09:02

chaitanya17 created an issue. See original summary.

Comment #2

chaitanya17 commented 10 January 2017 at 09:07

Status:

Active

» Needs review

Status	File	Size
new	patch-2842312.patch	3.62 KB

I have changed all $_POST variables with required \Drupal::request()->request->get method.

Please check attached working patch.

Comment #3

abhaysaraf commented 10 January 2017 at 19:44

Thanks Chaitanya17 for raising the concern and providing the solution.

Your solutions were very helpful.
I was not aware of this new way of handling $_POST data.

And in a solution for AnnotationsSaveAction.php, I have used:

$postReq = \Drupal::request()->request->all();
$nid = isset($postReq['nid']) ? $postReq['nid'] : FALSE;
$img_field = isset($postReq['img_field']) ? $postReq['img_field'] : FALSE;
$annotation = isset($postReq['annotation']) ? $postReq['annotation'] : FALSE;

This reduces multiple times calling get() function.

In 8.x-1.1, I have added a new feature and fixed the raised bug. Please use 8.x-1.1.