{"id":18508515,"date":"2025-06-12T06:17:43","date_gmt":"2025-06-12T06:17:43","guid":{"rendered":"https:\/\/wordpress.org\/support\/topic\/code-quality-16\/"},"modified":"2025-06-12T06:17:43","modified_gmt":"2025-06-12T06:17:43","slug":"code-quality-16","status":"publish","type":"topic","link":"https:\/\/wordpress.org\/support\/topic\/code-quality-16\/","title":{"rendered":"Code quality"},"content":{"rendered":"\n<ul>\n<li>You are using esc_url() in get_card_methods_from_api() incorrectly. Default context for esc_url() is display, meaning ensure safe url escaping and display in HTML, e.g. by replacing &amp; into &amp;amp;. Technically you could use URL encode, but URL you are constructing does not require any kind of encoding or escaping.<\/li>\n\n\n\n<li>Please switch to wp_remote_get()\/wp_remote_post()\/wp_remote_request() as alternative to curl. wp_remote_XXX() can be traced, logged and debugged. This in combination with query monitor \/ debug bar will also highlight outstanding performance bug I&#8217;ve reported 8 months ago <a href=\"https:\/\/wordpress.org\/support\/topic\/bug-in-get_card_method\/\">https:\/\/wordpress.org\/support\/topic\/bug-in-get_card_method\/<\/a>.<\/li>\n<\/ul>\n","protected":false},"template":"","class_list":["post-18508515","topic","type-topic","status-publish","hentry"],"jetpack_sharing_enabled":true,"_links":{"self":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic\/18508515","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic"}],"about":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/types\/topic"}],"version-history":[{"count":0,"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/topic\/18508515\/revisions"}],"wp:attachment":[{"href":"https:\/\/wordpress.org\/support\/wp-json\/wp\/v2\/media?parent=18508515"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}