It seems there are some issue because of megarow requests for refreshing the parent rows.
As they won't keep the destination, all the links from a refreshed row will have the destination the refresh megarow path.

There is a a Commerce Backoffice issue related : #2002008: Order operations destination changes after quick-edit save.

A solution is to pass the destination value to the refresh megarow request.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vasike’s picture

Status: Active » Needs review
FileSize
1.88 KB

and there is the patch that does it.

Artusamak’s picture

Status: Needs review » Needs work

What behavior do we expect? If i'm opening a megarow i'm not expecting to be redirected at some point.

vasike’s picture

Status: Needs work » Needs review

@Artusamak: could you please check the related issue (Commerce Backoffice bug) : #2002008: Order operations destination changes after quick-edit save.

Artusamak’s picture

Status: Needs review » Needs work

I did and it's not answering my question.

Artusamak’s picture

Status: Needs work » Closed (won't fix)
lmeurs’s picture

Status: Closed (won't fix) » Needs review
FileSize
1.62 KB
22.78 KB

@Artusamak: Take the operations menu in the screenshot for example. This menu does not only contain a Quick edit link that opens the megarow, but also links to other pages like the Edit and Delete links. The first link initially points at ie.:

admin/commerce/orders/13/edit?destination=admin/commerce/orders

but since the module does not preserve the initial destination parameter, after an AJAX submission the URL becomes:

admin/commerce/orders/13/edit?destination=views_megarow/refresh/commerce_backoffice_orders/admin_page

The problem is that after a quick edit all links in a megarow contain wrong destinations. When one clicks the Edit or Delete link, he will eventually get redirected to views_megarow/refresh/commerce_backoffice_orders/admin_page instead of admin/commerce/orders.

I adjusted vasike's patch to the latest dev, see attached.

Screenshot

lmeurs’s picture

The previous patch appended the destination parameter to the URL without respecting the optional Drupal.settings.ViewsMegarow.args arguments which were being appended on top of that. This is fixed in attached patch.

Artusamak’s picture

Status: Needs review » Fixed

OK, thanks for the patch it's committed.

  • Artusamak committed 609312b on 7.x-1.x authored by lmeurs
    Issue #2309117 by lmeurs, vasike: Pass/Keep the destination URL query...

Status: Fixed » Closed (fixed)

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