Download & Extend
Issues

minimal profile + seven leads to broken 'node/add' link

Posted by beejeebus on July 13, 2010 at 4:48am
Project:Drupal core
Version:7.x-dev
Component:Seven theme
Category:bug report
Priority:major
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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

Login or register to post comments

Comments

#1

Posted by beejeebus on July 13, 2010 at 4:49am

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

#2

Posted by beejeebus on July 13, 2010 at 5:26am
Title:minimal profile + stark leads to broken 'node/add' link» minimal profile + seven leads to broken 'node/add' link

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

AttachmentSizeStatusTest resultOperations
i-can-haz-content.jpg38.85 KBIgnored: Check issue status.NoneNone

#3

Posted by Jody Lynn on August 12, 2010 at 2:47am
Component:base system» Seven theme
Status:active» needs review

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

AttachmentSizeStatusTest resultOperations
852394.patch782 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 22,832 pass(es).View details

#4

Posted by reglogge on August 25, 2010 at 9:07am
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.

#5

Posted by Jeff Burnz on August 25, 2010 at 9:27am
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.

#6

Posted by reglogge on August 25, 2010 at 9:40am
Status:needs work» needs review

grrrrr...

AttachmentSizeStatusTest resultOperations
852394.patch772 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 26,201 pass(es).View details

#7

Posted by Jody Lynn on August 25, 2010 at 2:56pm
Status:needs review» reviewed & tested by the community

Dreditor to the rescue.

#8

Posted by sun on September 28, 2010 at 11:10pm

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

#9

Posted by Dries on October 3, 2010 at 12:10am
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.

#10

Posted by System Message on October 17, 2010 at 12:20am
Status:fixed» closed (fixed)

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

nobody click here