Skip to content

Add 'crypto' custom route issue #154 #156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
DarrenWestwood wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add 'crypto' custom route issue #154 #156

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
122a360
Add 'crypto' custom route issue #154
DarrenWestwood Oct 8, 2019
10b6b04
Flush rewrite rules on activation/deactivation
DarrenWestwood Oct 22, 2019
7611c39
Merge branch 'master' into custom-url-routes
DarrenWestwood Oct 28, 2019
9e7493a
update rewrite
DarrenWestwood Oct 28, 2019
c257013
Merge remote-tracking branch 'origin/custom-url-routes' into custom-u…
DarrenWestwood Oct 28, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions blockonomics-woocommerce.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,12 @@ function blockonomics_woocommerce_init()
add_action('woocommerce_email_customer_details', 'nolo_bnomics_woocommerce_email_customer_details', 10, 1);
add_filter('woocommerce_payment_gateways', 'woocommerce_add_blockonomics_gateway');
add_filter('clean_url', 'bnomics_async_scripts', 11, 1 );
add_action('init', 'custom_rewrite_rules');

/* Custom Rewrite Rule */
function custom_rewrite_rules() {
Copy link
Owner

@blockonomics blockonomics Oct 10, 2019

Choose a reason for hiding this comment

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

Let us show_order . Also don't hardcode wc-api=WC_Gateway_Blockonomics let us use WC()->api_request_url else it won't work for WC < 3.0

Copy link
Collaborator Author

@DarrenWestwood DarrenWestwood Oct 28, 2019

Choose a reason for hiding this comment

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

We cannot use WC()->api_request_url because add_rewrite_rule requires the url to start with index.php. All wordpress urls can be accessed in this manner.
In later versions of woocommerce, they added their own rewrite rules to change the url, however, we still need to access the original page with the form index.php?wc-api=WC_Gateway_Blockonomics

add_rewrite_rule( 'crypto/?$', 'index.php?wc-api=WC_Gateway_Blockonomics', 'top' );
}

/**
* Add this Gateway to WooCommerce
Expand Down
3 changes: 2 additions & 1 deletion php/WC_Gateway_Blockonomics.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,8 @@ public function process_payment($order_id)
update_option('blockonomics_orders', $blockonomics_orders);
$order_url = WC()->api_request_url('WC_Gateway_Blockonomics');
$order_url = add_query_arg('show_order', $address, $order_url);

$order_url = str_replace('wc-api/WC_Gateway_Blockonomics', 'crypto', $order_url);
Copy link
Owner

@blockonomics blockonomics Oct 10, 2019

Choose a reason for hiding this comment

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

there is no need to first add WC()->api_request_url and then replace it. We should directly use show_order

Copy link
Collaborator Author

@DarrenWestwood DarrenWestwood Oct 28, 2019

Choose a reason for hiding this comment

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

We do not have any way to directly access show_order as we are not creating any page or post type. We still have to use WC()->api_request_url('WC_Gateway_Blockonomics') to fetch the correct url.


update_post_meta($order_id, 'blockonomics_address', $address);

return array(
Expand Down