Closed calebjenkins closed 12 years ago
Did you commit the changes to the WordPress plugin? It looks the same to me...
I went ahead and made some updates to it - thanks for the kick in the pants and all the hard work on refactoring! I will make sure to alert the original developer to your changes.
I'm pretty sure I did commit the changes to the WP plugin. I'll double check later today.
I made significant refactorings. Moved common values in to an AppConfig class that can be passed from page to page in stread of stuffing everything in to the Querystring.
Moved the networking interactions in to their own set of classes and out of the page behinds. This reduced a significant amount of duplication and makes for more testable classes down the road.
Updated the Wordpress plug in to support an (optional) user_id parameter for scenarios when the blog author is not user ID 0, and also usefull when WordPress is running with multiple instances.
Also, wrapped a majority of the content in [[Data]] tags to reduce the number of things that XDocument.Parse was choaking on.
Note: You'll want to update the Wordpress Plug in when using this branch. You can use the old plugin, but some content (from the main screen) will come out incorrectly formated.