{"action":"created","issue":{"url":"https://api.github.com/repos/woothemes/woocommerce/issues/11355","repository_url":"https://api.github.com/repos/woothemes/woocommerce","labels_url":"https://api.github.com/repos/woothemes/woocommerce/issues/11355/labels{/name}","comments_url":"https://api.github.com/repos/woothemes/woocommerce/issues/11355/comments","events_url":"https://api.github.com/repos/woothemes/woocommerce/issues/11355/events","html_url":"https://github.com/woothemes/woocommerce/issues/11355","id":163756808,"number":11355,"title":"WC_Abstract_Order->get_cancel_order_url() returning invalid URI","user":{"login":"aianus","id":1322087,"avatar_url":"https://avatars.githubusercontent.com/u/1322087?v=3","gravatar_id":"","url":"https://api.github.com/users/aianus","html_url":"https://github.com/aianus","followers_url":"https://api.github.com/users/aianus/followers","following_url":"https://api.github.com/users/aianus/following{/other_user}","gists_url":"https://api.github.com/users/aianus/gists{/gist_id}","starred_url":"https://api.github.com/users/aianus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aianus/subscriptions","organizations_url":"https://api.github.com/users/aianus/orgs","repos_url":"https://api.github.com/users/aianus/repos","events_url":"https://api.github.com/users/aianus/events{/privacy}","received_events_url":"https://api.github.com/users/aianus/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-07-05T01:23:43Z","updated_at":"2016-07-06T00:20:37Z","closed_at":"2016-07-05T02:34:35Z","body":"## EXPLANATION OF THE ISSUE\r\n\r\nThe [get_cancel_order_url](https://docs.woothemes.com/wc-apidocs/source-class-WC_Abstract_Order.html#2137-2155) function on a fresh install is returning an invalid URI (in my case, `http://coinbase-woocommerce-testing.alexian.us/cart/?cancel_order=true&#038;order=wc_order_577b0453cd8b5&#038;order_id=9&#038;redirect`)\r\n\r\n## STEPS TO REPRODUCE THE ISSUE\r\n\r\n1. Install Wordpress 4.5.3\r\n2. Install WooCommerce Version 2.6.2\r\n3. Install a custom payment gateway, for example, [coinbase-woocommerce](https://github.com/coinbase/coinbase-woocommerce)\r\n4. When the function [get_cancel_order_url](https://docs.woothemes.com/wc-apidocs/source-class-WC_Abstract_Order.html#2137-2155) is called on the order as part of the checkout process, it returns an invalid URI, causing Coinbase's backend to reject the checkout.\r\n\r\nThe URI that the plugin is getting from that function is `http://coinbase-woocommerce-testing.alexian.us/cart/?cancel_order=true&#038;order=wc_order_577b0453cd8b5&#038;order_id=9&#038;redirect`\r\n\r\nThe expected URI, and the URI that was returned by previous versions of wordpress/woocommerce, is  `http://coinbase-woocommerce-testing.alexian.us/cart/?cancel_order=true&order=wc_order_577b0453cd8b5&order_id=9&redirect` or similar (no redundant encoded ampersands).\r\n\r\n## SYSTEM STATUS REPORT\r\n\r\n```\r\n`\r\n### WordPress Environment ###\r\n\r\nHome URL: http://coinbase-woocommerce-testing.alexian.us\r\nSite URL: http://coinbase-woocommerce-testing.alexian.us\r\nWC Version: 2.6.2\r\nLog Directory Writable: ✔\r\nWP Version: 4.5.3\r\nWP Multisite: –\r\nWP Memory Limit: 256 MB\r\nWP Debug Mode: ✔\r\nWP Cron: ✔\r\nLanguage: en_US\r\n\r\n### Server Environment ###\r\n\r\nServer Info: Apache/2.4.7 (Ubuntu)\r\nPHP Version: ❌ 5.5.9-1ubuntu4.17 - We recommend a minimum PHP version of 5.6. See: How to update your PHP version\r\nPHP Post Max Size: 8 MB\r\nPHP Time Limit: 30\r\nPHP Max Input Vars: 1000\r\ncURL Version: 7.35.0\r\nOpenSSL/1.0.1f\r\n\r\nSUHOSIN Installed: –\r\nMax Upload Size: 2 MB\r\nDefault Timezone is UTC: ✔\r\nfsockopen/cURL: ✔\r\nSoapClient: ✔\r\nDOMDocument: ✔\r\nGZip: ✔\r\nMultibyte String: ✔\r\nRemote Post: ✔\r\nRemote Get: ✔\r\n\r\n### Database ###\r\n\r\nWC Database Version: 2.6.2\r\n: \r\nwoocommerce_sessions: ✔\r\nwoocommerce_api_keys: ✔\r\nwoocommerce_attribute_taxonomies: ✔\r\nwoocommerce_downloadable_product_permissions: ✔\r\nwoocommerce_order_items: ✔\r\nwoocommerce_order_itemmeta: ✔\r\nwoocommerce_tax_rates: ✔\r\nwoocommerce_tax_rate_locations: ✔\r\nwoocommerce_shipping_zones: ✔\r\nwoocommerce_shipping_zone_locations: ✔\r\nwoocommerce_shipping_zone_methods: ✔\r\nwoocommerce_payment_tokens: ✔\r\nwoocommerce_payment_tokenmeta: ✔\r\nMaxMind GeoIP Database: ❌ The MaxMind GeoIP Database does not exist - Geolocation will not function. You can download and install it manually from http://dev.maxmind.com/geoip/legacy/geolite/ to the path:\r\n\r\n### Active Plugins (2) ###\r\n\r\ncoinbase-woocommerce: by Coinbase Inc. – 2.1.2\r\nWooCommerce: by WooThemes – 2.6.2\r\n\r\n### Settings ###\r\n\r\nForce SSL: –\r\nCurrency: CAD ($)\r\nCurrency Position: left\r\nThousand Separator: ,\r\nDecimal Separator: .\r\nNumber of Decimals: 2\r\n\r\n### API ###\r\n\r\nAPI Enabled: ✔\r\n\r\n### WC Pages ###\r\n\r\nShop Base: #4 - /shop/\r\nCart: #5 - /cart/\r\nCheckout: #6 - /checkout/\r\nMy Account: #7 - /my-account/\r\n\r\n### Taxonomies ###\r\n\r\nProduct Types: external (external)\r\ngrouped (grouped)\r\nsimple (simple)\r\nvariable (variable)\r\n\r\n\r\n### Theme ###\r\n\r\nName: Twenty Sixteen\r\nVersion: 1.1 – 1.2 is available\r\nAuthor URL: https://wordpress.org/\r\nChild Theme: ❌ – If you're modifying WooCommerce on a parent theme you didn't build personally\r\nthen we recommend using a child theme. See: How to create a child theme\r\n\r\nWooCommerce Support: ✔\r\n\r\n### Templates ###\r\n\r\nOverrides: –\r\n````\r\n\r\n<!--\r\nBEFORE POSTING YOUR ISSUE\r\n- These comments won't show up when you submit the issue.\r\n- Try to add as much detail as possible. Be specific!\r\n- GitHub issues ARE NOT FOR SUPPORT! If you have questions, use the either:\r\n  - https://wordpress.org/support/plugin/woocommerce for general support\r\n  - https://support.woothemes.com for premium plugin support\r\n- If you're requesting a new feature, explain why you'd like it to be added.\r\n- Search this repository (top of the page) for the issue and it has not been fixed or reported already.\r\n\r\nBefore logging BUGS, ensure you:\r\n- Use the latest stable release of WooCommerce.\r\n- Disabled all plugins to ensure it's a core bug and not a plugin issue.\r\n- Switched to Twenty Twelve theme to ensure it's a core bug and not a theme issue.\r\n-->\r\n"},"comment":{"url":"https://api.github.com/repos/woothemes/woocommerce/issues/comments/230640165","html_url":"https://github.com/woothemes/woocommerce/issues/11355#issuecomment-230640165","issue_url":"https://api.github.com/repos/woothemes/woocommerce/issues/11355","id":230640165,"user":{"login":"claudiosmweb","id":1264099,"avatar_url":"https://avatars.githubusercontent.com/u/1264099?v=3","gravatar_id":"","url":"https://api.github.com/users/claudiosmweb","html_url":"https://github.com/claudiosmweb","followers_url":"https://api.github.com/users/claudiosmweb/followers","following_url":"https://api.github.com/users/claudiosmweb/following{/other_user}","gists_url":"https://api.github.com/users/claudiosmweb/gists{/gist_id}","starred_url":"https://api.github.com/users/claudiosmweb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/claudiosmweb/subscriptions","organizations_url":"https://api.github.com/users/claudiosmweb/orgs","repos_url":"https://api.github.com/users/claudiosmweb/repos","events_url":"https://api.github.com/users/claudiosmweb/events{/privacy}","received_events_url":"https://api.github.com/users/claudiosmweb/received_events","type":"User","site_admin":false},"created_at":"2016-07-06T00:20:37Z","updated_at":"2016-07-06T00:20:37Z","body":"@aianus works when printing on HTML. Several plugins uses it and on Core too. I need to confess that I prefer to escape only before to print and not ever that we try to get the URL, but this is a legacy method, so no good try to change it now =/"}}
{"actor":{"display_login":"claudiosmweb"}}