Multiple posts with same post-slug

Patch submitted for WP Patch Day :D

File attached (as .txt file this time!)

Sorry, I can't provide a diff because I knocked this up in an internet cafe ;D The file itself should be pretty self explanatory though...

I chose option b) - automatically creating a secondary name. I don't think there's any point in bothering the user with this kind of thing. Plus anything they entered would have to be re-sanitised and checked... It's a whole lot quicker to just do it 'magically' :D

Tested and working on my own WP installation, and I've updated it to use $wpdb->posts instead of $tableposts as per the latest CVS, so there should be no problems.

Contact address in the text file if there are any problems / comments.

But surely you want to be able to post multiple posts with the same slug, so long as they're on different days (and thus get a different permalink under the default settings) why should it matter?

Because I don't want to use dates in the permalinks at all. :)

Actually, there is a slight problem with version 1.01 of the code I submitted. At the moment, the name is being tested even when it is a draft. Seeing as you may or may not publish a draft, it doesn't make sense to alter the post_name until it is published. Version 2 corrects this, but requires that the code be in the 'post' section and the 'editpost' section. As identical code is called twice, maybe it makes sense to have this as a separate function?

ie: $post_name = get_unique_postname($post_name);

Uploading an attachment with the alterations now... Sorry no diff again :( (But you should get the idea).

Another minor change, not in the code this time, but of the position where the code should be inserted. Full explanation in ​http://www.jamietalbot.com/wp-hacks/post-slug-patch-103.phps but rough explanation is that problems would occur if a post was continually edited and saved but not published. Moving the code further down the file fixes it.

This also highlights what I consider to be a significant UI bug - the existence of a "Draft", "Private" and "Publish" option group which is then overridden by which button the user clicks. It was because of this that I didn't notice the bug until now, as I was clicking Draft and then pushing Publish by mistake. For half of the file, post-status is treated as 'draft' until a test half way down where it changes to 'publish'. Not only is it redundant, it is also confusing (to me at least) both as a programmer and blogger.

This seperate issue has previously been raised as bug 0000124, but seems to have got a little lost...

edited on: 08-02-04 15:19

edited on: 08-04-04 10:55

edited on: 08-04-04 10:56

Another small change, this time so the post doesn't match against itself. Bit more rigorously tested this time, so there shouldn't be any more...

​http://www.jamietalbot.com/wp-hacks/post-slug-patch-104.phps

This works for me, please close.

TODO: Use WP_Query to check against permalink structure.