I think it's currently possible that the address is overwritten on the order edit page by the address of the currently logged in admin. Is that right or is it just a bug in my specific case?

Perhaps hook_field_attach_form should check for " && arg(0) != 'admin'" in the if condition so that it does not apply there?

CommentFileSizeAuthor
#2 do_not_override_admin_order_page-2185425-2.patch925 bytesAnybody
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jerenus’s picture

Jerenus CreditAttribution: Jerenus commented
Status: Active » Needs work

Patch needed.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs work » Needs review
FileSize
do_not_override_admin_order_page-2185425-2.patch925 bytes

Patch attached. I'm using this in production since a long period of time already. Ready for review and to be applied. :)

Status: Needs review » Needs work

The last submitted patch, 2: do_not_override_admin_order_page-2185425-2.patch, failed testing.

The last submitted patch, 2: do_not_override_admin_order_page-2185425-2.patch, failed testing.

The last submitted patch, 2: do_not_override_admin_order_page-2185425-2.patch, failed testing.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs work » Reviewed & tested by the community

The patch is OK, there are simply no tests specified for the project yet.

  • Jerenus committed ad01377 on 7.x-1.x authored by Anybody 
    Issue #2185425 by Anybody, Jerenus: Ensure that the address is not...
Jerenus’s picture

Jerenus CreditAttribution: Jerenus commented
Status: Reviewed & tested by the community » Fixed

Thank you for your patch. Committed ad01377.

Jerenus’s picture

Jerenus CreditAttribution: Jerenus commented

Status: Fixed » Closed (fixed)

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