[theme-reviewers] Fwd: [WordPress Themes] #3553: THEME: The Common Blog - 1.0
Chip Bennett
chip at chipbennett.net
Wed Apr 27 13:05:27 UTC 2011
Suggestion?
Yes: don't name Post Titles as numbers-only, when using a date-based
permalink structure. :)
Chip
On Wed, Apr 27, 2011 at 5:55 AM, Taryn Wallis <taryn at phenomenoodle.com>wrote:
> Hi again Chip (and others!)
>
> Another comment re the issue with the post ID for the post with no title -
> any suggestions?
>
>
> #3553: THEME: The Common Blog - 1.0
> ---------------------------+----------------------------------
> Reporter: plebeian.se | Owner: phenomenoodle
> Type: theme | Status: closed
> Resolution: not-approved | Keywords: theme-the-common-blog
> ---------------------------+----------------------------------
>
> Comment (by feedmeastraycat):
>
> Hello
>
> I'm a friend of Mark and helped him locate this ... thing. That I still
> think is a bug. :)
>
> 1. Create a regular post created on 31th of March 2011 with title "Test"
> (gets slug "test")
> 2. Create another one on 14th of March 2011, but with title "31" (gets
> slug "31")
> 3. Create another one on 14th of March 2011, but with title "32" (gets
> slug "32")
> 4. Make sure permalinks is Month and name
>
> - The link for the first post gets /2011/03/test/ and works fine
> - The link for the second post gets /2011/03/31/ and shows a day archive
> for 31st of march 2011, which is the first post (if no posts exist on this
> date it gets 404, this is why this test case creates the first post on
> that specific date)
> - The link for the third post gets /2011/03/32/ and gives a 404
>
> The breaking point for the slug seams to be 100. Cause if you edit the
> slug of post 3 to 100 it gets /2011/03/100/ and works fine. But 99 gives
> error.
>
> If you put slug "0" you get /2011/03/0/ and you get month archive.
>
> This test puts in 31 as a slug manually. But the error is be the same if
> you have a post id 31 and put no title. Cause a post id 31 without title
> gets 31 as a slug. For example if you forget to put a title on your first
> post it will get /2011/03/1/ and shows a 404 (if it isn't the 1st when you
> post it in which case it shows a day archive for that day and shows that
> post) :)
>
> As long as the slug contains a letter this doesn't matter.
>
>
> Maybe I should put this into trac.wordpress.org?
>
>
> Regards,
> David
>
> --
> Ticket URL: <http://themes.trac.wordpress.org/ticket/3553#comment:8>
> ___________________________________________________________________
>
>
>
>
>
> On Tue, Apr 26, 2011 at 9:03 PM, Taryn Wallis <taryn at phenomenoodle.com>wrote:
>
>> Ah, I see. Thanks, Chip!
>> ___________________________________________________________________
>>
>> *Taryn Wallis*
>> Phenomenoodle
>> WordPress website design & development for small businesses with big
>> dreams
>> www.phenomenoodle.com
>> +44 1875 34 0044
>> +44 774 839 8473
>>
>>
>>
>>
>> On Tue, Apr 26, 2011 at 8:45 PM, Chip Bennett <chip at chipbennett.net>wrote:
>>
>>> It's not really a "bug". It is a result of importing the test data. On
>>> the source site (wpthemetestdata.wordpress.com), the PostID for that
>>> post is "14". However, the Post ID changes upon import. The solution is to
>>> delete the post slug, and to let WordPress re-generate it.
>>>
>>> I will add this clarification to the list of Codex changes I need to
>>> make.
>>>
>>> Chip
>>>
>>> On Tue, Apr 26, 2011 at 2:42 PM, Taryn Wallis <taryn at phenomenoodle.com>wrote:
>>>
>>>> In one of the themes I reviewed (
>>>> http://themes.trac.wordpress.org/ticket/3553<http://themes.trac.wordpress.org/ticket/3553#comment:6>
>>>> ), I noted that the post with no title wasn't showing up in the blog
>>>> and gave a 404 error when I tried to access it directly (via the 'view post'
>>>> link in the Posts screen).
>>>>
>>>> The theme author has written the response below detailing a bug he found
>>>> - has anyone come across this before?
>>>>
>>>> Thanks for your help!
>>>> T
>>>>
>>>>
>>>> ---------- Forwarded message ----------
>>>> From: WordPress Themes <theme-reviewers at lists.wordpress.org>
>>>> Date: Tue, Apr 26, 2011 at 7:40 PM
>>>> Subject: Re: [WordPress Themes] #3553: THEME: The Common Blog - 1.0
>>>> To:
>>>>
>>>>
>>>> #3553: THEME: The Common Blog - 1.0
>>>> ---------------------------+----------------------------------
>>>> Reporter: plebeian.se | Owner: phenomenoodle
>>>> Type: theme | Status: closed
>>>> Resolution: not-approved | Keywords: theme-the-common-blog
>>>> ---------------------------+----------------------------------
>>>>
>>>> Comment (by plebeian.se):
>>>>
>>>> Regarding "(no title) post not displayed and gives 404 error", I
>>>> switched
>>>> to Twentyten and noticed that it had the same error. After some testing
>>>> I
>>>> discovered that if you set your permalink structure to "Month and
>>>> name",
>>>> and you have a low post id, WordPress takes the post id as a date.
>>>>
>>>> Example:
>>>> An empty post with id number 14, posted on the 4th march 2011, will get
>>>> permalink /2011/03/14/, if you have "Month and name".
>>>> This will give a 404 if you have no posts posted on the 14th of march
>>>> 2011. And it will give an "day archive" for the 14th of march 2011 if
>>>> you
>>>> have any other posts on that specific date.
>>>> If you change this to "Day and name" instead the post permalink will be
>>>> /2011/03/04/14/ and the post is found.
>>>>
>>>> Ive changed my common settings to "Day and name" and you can see that
>>>> it
>>>> works here:
>>>> http://www.plebeian.se/dev/2008/05/04/14/
>>>>
>>>> And like I said, Twentyten has the exact same "bug".
>>>>
>>>> If the post ID is bigger, like 403 or something, this problem doesn't
>>>> exist. Basically, I think it's not a problem as long as the post id is
>>>> bigger then the number of dates in the specific month of that post. A
>>>> post
>>>> id 31 will work if posted in February. But not in march.
>>>>
>>>>
>>>> Do I still need to fix this? :)
>>>>
>>>> --
>>>> Ticket URL: <http://themes.trac.wordpress.org/ticket/3553#comment:6>
>>>> WordPress Themes <http://themes.trac.wordpress.org/>
>>>> WordPress.org Theme Directory Reviews
>>>>
>>>>
>>>> _______________________________________________
>>>> theme-reviewers mailing list
>>>>
>>>> theme-reviewers at lists.wordpress.org
>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>
>>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110427/e4ef37ff/attachment.htm>
More information about the theme-reviewers
mailing list