1. clean install with minimal profile

2. change to stark as default theme

3. go to 'node/add' and look for "You have not created any content types yet. Go to the content type creation page to add a new content type."

4. FAIL

CommentFileSizeAuthor
#6 852394.patch772 bytesreglogge
#3 852394.patch782 bytesjody lynn
#2 i-can-haz-content.jpg38.85 KBAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented

once #323477: Increase simpletest speed by running on a simplified profile lands, we can write tests that catch this.

Anonymous’s picture

Anonymous (not verified) commented
Title: minimal profile + stark leads to broken 'node/add' link » minimal profile + seven leads to broken 'node/add' link
StatusFileSize
new i-can-haz-content.jpg38.85 KB

woops, that should be "seven", not "stark". screenshot of 'node/add' attached.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Component: base system » Seven theme
Status: Active » Needs review
StatusFileSize
new 852394.patch782 bytes

Here's the fix. seven_node_add_list was missing that part.

reglogge’s picture

reglogge commented
Priority: Normal » Major
Status: Needs review » Reviewed & tested by the community

Reviewed and tested this and closed #885518: No warning message present on node/add if content types not added as a duplicate of this issue.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Reviewed & tested by the community » Needs work 
+++ themes/seven/template.php	12 Aug 2010 02:43:29 -0000
@@ -47,6 +47,9 @@ function seven_node_add_list($variables)
+  }          ¶
   return $output;

has white space

Powered by Dreditor.

reglogge’s picture

reglogge commented
Status: Needs work » Needs review
StatusFileSize
new 852394.patch772 bytes

grrrrr...

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Dreditor to the rescue.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

#6: 852394.patch queued for re-testing.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

It would be nice if we could get rid of seven_node_add_list() so we wouldn't need to duplicate this code.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.