Opened 11 years ago
Closed 11 years ago
#21186 closed defect (bug) (fixed)
WP_Rewrite::mod_rewrite_rules() should use site_url()
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | normal | Version: | 3.4.1 |
Component: | Rewrite Rules | Keywords: | has-patch dev-feedback |
Focuses: | Cc: |
Description
I could be off here, but I'm thinking site_url() should be used here in place of get_option( 'siteurl' ). The potential issues are naturally inferred, and slightly further down in the function, home_url() is used.
Let me know if I'm missing something here.
Attachments (1)
Change History (4)
Note: See
TracTickets for help on using
tickets.
It's been using get_option()/get_setting() forever. There doesn't seem to be a reason is hasn't changed to site_url(). mod_rewrite_rules() is scheme agnostic so the patch should be fine.