On deleting an app, the confirm form does not work correctly, as the parameters for redirecting needed updating to match D7's drupal_goto signature, and the app info was not pulled in correctly to the form. The attached patch fixes these problems :-)

CommentFileSizeAuthor
fb-app-deletion-errors.patch819 bytesjames.williams

Comments

geerlingguy’s picture

geerlingguy commented

Closed #1095874: Removing app returns nasty error in favor of this issue, as it contains a patch. Patch looks good from a quick visual inspection, but I didn't test it.

beatnikdude’s picture

beatnikdude commented
Title: Errors on deleting an app » Errors on deleting FaceBook Application from Drupal
Status: Needs review » Reviewed & tested by the community

When trying to remove a FB application @ Home » Administration » Structure » Drupal for Facebook via Edit & Delete, the application is not removed and I receive the error:
Recoverable fatal error: Argument 2 passed to drupal_goto() must be an array, string given in drupal_goto()...

I have applied and tested this patch on 7.x-3.x-dev, it is simple and fixes the above error.
-Thanks

Dave Cohen’s picture

Dave Cohen commented
Status: Reviewed & tested by the community » Fixed

Thanks! Checked in.

Status: Fixed » Closed (fixed)

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