When a user tries to delete a user manifest like user-manifest by going at link: admin/people/manifest/user-manifest/delete. They encounter this error because at that moment destination key is not defined in $_GET variable.

Comments

xangy created an issue. See original summary.

xangy’s picture

Assigned: xangy » Unassigned
Status: Active » Needs review
StatusFileSize
new957 bytes

Added a patch. In this patch, I checked if destination key exists in $_GET variable or not. If so, is it empty or not?

xangy’s picture

Issue summary: View changes
gedur’s picture

Status: Needs review » Needs work

I think it is better to just check with empty():

!empty($_GET['destination']) ? $_GET['destination'] : 'admin/people/manifest',

Could you redo the patch to commit it?

gedur’s picture

Status: Needs work » Fixed

I've already done it (the destination doesn't need to be specified as drupal takes it by itself), but I've attributed to you the commit, thanks for your support

  • GeduR committed 18b7132 on 7.x-1.x authored by xangy
    Issue #2731363 by xangy, GeduR: Undefined index: destination in...

Status: Fixed » Closed (fixed)

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