We need a way to add data from dibs_transactions table into Commerce module view.

CommentFileSizeAuthor
#4 dibs-views-integration-with-commerce-module-2202233-4.patch4.61 KBddod
#3 dibs-views-integration-with-commerce-module-2202233-3.patch2.13 KBddod
#2 dibs-views-integration-with-commerce-module-2202233-2.patch2.7 KBddod

Comments

beltofte’s picture

beltofte
Primary language Danish
Location Copenhagen 🇩🇰
commented

Feel free to work on it and upload a patch :-) The current maintainers are lacking time to work on it, so it will not happend in the near future without other developers stepping in.

ddod’s picture

ddod commented
StatusFileSize
new dibs-views-integration-with-commerce-module-2202233-2.patch2.7 KB

Done.

I used hook_views_api and hook_views_data to expose payment_order_id and payment_transaction_id columns from dibs_transactions to Commerce module views. The join is performed on commerce_payment_transaction.remote_id using dibs_transactions.tid.

Other columns can be exposed using the same pattern as needed.

Cheers.

ddod’s picture

ddod commented
StatusFileSize
new dibs-views-integration-with-commerce-module-2202233-3.patch2.13 KB

**ignore - better solution in the next comment
A patch for commerce_payment_ui_views_default_views() using the alter function to add the already exposed columns from the previous patch to the default Commerce Payment View.

ddod’s picture

ddod commented
StatusFileSize
new dibs-views-integration-with-commerce-module-2202233-4.patch4.61 KB

I combined the two patches in this one, also commerce_payment_ui_views_default_views_alter() was moved to dibs.views.inc.

boyan.borisov’s picture

boyan.borisov commented
Status: Active » Needs review
googletorp’s picture

googletorp commented
Status: Needs review » Fixed

I didn't add all of it, I don't feel we should alter commerce views by default when installing the dibs module.

  • googletorp committed de2e5bd on 
    Issue #2202233 by googletorp, ddod: Add initial work with Views...

FrittenKeeZ’s picture

FrittenKeeZ commented

Out of curiosity, why have the file dibs-views-integration-with-commerce-module-2202233-4.patch been added to the project?

Status: Fixed » Closed (fixed)

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