[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