search

tamagokun / pomander-wordpress

Wordpress tasks for use with Pomander
http://ripeworks.com/pomander/wordpress.html
MIT License
36 stars 5 forks source link

Replaced URLs are being duplicated #3

Closed asanger closed 11 years ago

asanger commented 11 years ago

Check it out: http://blfd.levelsetsolutions.com/ www.bluelinefd.com becomes http://blfd.levelsetsolutions.com/blfd.levelsetsolutions.com/

asanger commented 11 years ago

After reviewing the values in the database, I found that this occurs when the home option doesn't contain "http://".

tamagokun commented 11 years ago

This is strange. If the home option doesn't contain http://, then it shouldn't get changed when looking at the sed command: https://github.com/tamagokun/pomander-wordpress/blob/master/lib/tasks.php#L79

asanger commented 11 years ago

hmmm... well, you replaced the old sed with a php preg_replace, though that shouldn't cause any issues. It could be an issue related to issue #2 combined with the directory structure that pomander uses, though at that point this wouldn't really be a pomander issue as much as a pre-pomander wordpress issue. I'll run the original blueline project through a cold deploy tomorrow to see if I can replicate it.

On Tue, Dec 18, 2012 at 9:17 PM, Mike Kruk notifications@github.com wrote:

This is strange. If the home option doesn't contain http://, then it shouldn't get changed when looking at the sed command: https://github.com/tamagokun/pomander-wordpress/blob/master/lib/tasks.php#L79

— Reply to this email directly or view it on GitHubhttps://github.com/tamagokun/pomander-wordpress/issues/3#issuecomment-11514812.

Alec Sanger 248.941.3813

asanger commented 11 years ago

Okay - I figured it out.

https://github.com/tamagokun/pomander-wordpress/blob/master/lib/tasks.php#L79 This line does a replace on all URLs that are after "http://". That means that the value it replaces with must leave "http://" off.

https://github.com/tamagokun/pomander-wordpress/blob/master/lib/tasks.php#L81 This line updates the value of siteurl and home in wp_options, however it doesn't add an "http://".

All that I have to do to correct the issue of duplicate URIs is update the "home" record by adding "http://" before the URL.