Closed MdAsifHossainNadim closed 2 months ago
The changes improve URL handling and output security in two template files by introducing and utilizing $home_url
and $custom_store_url
variables. Additionally, esc_url
is used to sanitize URLs, and printf
is applied for regulated display of HTML content, enhancing maintainability and readability.
Files | Change Summary |
---|---|
.../account/update-customer-to-vendor.php |
Initialized $home_url and $custom_store_url ; used esc_url ; replaced echo with printf |
.../global/seller-registration-form.php |
Introduced $home_url and $custom_store_url for streamlined URL handling in the shop URL input |
Objective | Addressed | Explanation |
---|---|---|
Disabled order editing option when the order status is cancelled or refunded (#59) | ❓ | No visible changes related to this issue. |
In code we tread with care and pride,
URLs secure, no bugs to hide.
Echo replaced, conditions clear,
esc_url
ensures no fear.
Our template's now a stronger shield,
With clarity and clean code wield.
코드랩비트, onward we cheer! 🐰✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Scenario: Double slash is rendering when using WPML on the vendor registration form
Steps to reproduce:
Screenshot:
Scenario: Double slash is rendering when using WPML on the vendor registration form created via dokan shortcode
Steps to reproduce:
[dokan-vendor-registration]
Screenshot:
@shashwatahalder01 Brother, kindly let me know if any inconsistencies found.
@shashwatahalder01 @MdAsifHossainNadim does not work, please review test here: https://github.com/getdokan/dokan-wpml/issues/59#issuecomment-2233600772
All Submissions:
Changes proposed in this Pull Request:
Related Pull Request(s)
Closes
How to test the changes in this Pull Request:
Changelog entry
Fix: Shop URL rendered double slash when using WPML on vendor registration.
Detailed Description of the pull request. What was previous behaviour and what will be changed in this PR.
Before Changes
Go throw the issue details: Link
After Changes
Feature Video (optional)
Link of detailed video if this PR is for a feature.
PR Self Review Checklist:
FOR PR REVIEWER ONLY:
Summary by CodeRabbit
New Features
Improvements
esc_url
for URL concatenation.printf
instead ofecho
for displaying links.