Skip to content

Issue #2807559: Add owner filter to order/cart overview#553

Open
cornifex wants to merge 1 commit intodrupalcommerce:8.x-2.xfrom
cornifex:2807559-filter-by-owner
Open

Issue #2807559: Add owner filter to order/cart overview#553
cornifex wants to merge 1 commit intodrupalcommerce:8.x-2.xfrom
cornifex:2807559-filter-by-owner

Conversation

@cornifex
Copy link
Copy Markdown
Contributor

No description provided.

exposed: true
expose:
operator_id: combine_op
label: Owner
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be "Customer" instead of "Owner"

destination: true
entity_type: commerce_order
plugin_id: entity_operations
name:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you add this? We already have a Customer column that shows the user name.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to add it to use in the combined filter, otherwise it would only search by UID

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bojanz we'll only be able to search by contact mail without it. The combined filters filter requires fields in the "Fields" section.

@cornifex cornifex force-pushed the 2807559-filter-by-owner branch 2 times, most recently from fa2c522 to b4009d9 Compare November 16, 2016 20:03
@mglaman
Copy link
Copy Markdown
Contributor

mglaman commented Nov 16, 2016

This needs codestyle fix. Newline.

FILE: ...alcommerce/commerce/modules/order/commerce_order.post_update.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 86 | ERROR | [x] Expected 1 newline at end of file; 0 found
    |       |     (Drupal.Files.EndFileNewline.NoneFound)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
Time: 5.88 secs; Memory: 36Mb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants