Access denied page shown after submitting form that creates a unpublished node

Can we also have a msg, that informs the user what happens? I would be befuddled, if I am redirected - and unable to navigate to the page I just created. Something a long the lines of "You created *****, but you do not have sufficient permissions to view the unpublished ***"

Hi.

I created a patch for this issue but i have a doubt, let's suppose to the user not have permissions to view their own content and be redirected to the edit page of that node, so, what happen if the user neither have permissions to edit their own content?, where suppose to be redirected?

I suppose to the user should be redirected to the page where he came from? if the answer is positive just let me know and i will change my patch.

For now i'm sure to my patch still needs work.

This patch redirects users to is they don't have access to the node. That's the same thing that happens when they delete a node.

+++ b/core/modules/node/node.testundefined
@@ -573,6 +573,25 @@ class NodeCreationTestCase extends NodeWebTestCase {
+   * Create an unpublished node and confirm correct redirect behavior.

s/Create/Creates

+++ b/core/modules/node/node.testundefined
@@ -573,6 +573,25 @@ class NodeCreationTestCase extends NodeWebTestCase {
+    $langcode = LANGUAGE_NOT_SPECIFIED;
+    $edit["title"] = $this->randomName(8);
+    $edit["body[$langcode][0][value]"] = $this->randomName(16);

No real reason for a new langcode variable.

+++ b/core/modules/node/node.testundefined
@@ -573,6 +573,25 @@ class NodeCreationTestCase extends NodeWebTestCase {
+    $this->assertText(t('Welcome to Drupal'), t('The user is redirected to the home page.'));

The assertion success message shouldn't use t().

I rerolled for the new test file structure.

@tim your first and third comments above aren't consistent with what's being done in the existing tests in NodeCreationTest.php so I didn't make those changes.

Can;t put a constant inside double quotes...

+++ b/core/modules/node/lib/Drupal/node/Tests/NodeCreationTest.php
@@ -86,4 +86,22 @@ class NodeCreationTest extends NodeTestBase {
+    variable_set('site_frontpage', 'node');
+

Just a note to myself and sun. If this patch is committed before #1496542: Convert site information to config system then we'll have to convert this new variable to the new config object.

Otherwise RTBC

I updated the patch so that the test uses the new config object to set the system front page as mentioned in comment #16

Thanks @nrambeck, looks great.

+++ b/core/modules/node/lib/Drupal/node/Tests/NodeCreationTest.phpundefined
@@ -86,4 +86,23 @@ class NodeCreationTest extends NodeTestBase {
+    // Set the the font page to the default "node" page

This comment could be cleaned up a bit:
Set the front page to the default "node" page.

Leaving at needs review.

I cleaned up nrambeck's comment and moved the redirect line to the new NodeFormController file.

This makes a lot of sense. Thanks for the test!

Committed and pushed to 8.x. Marking for D7 backport.

Backported #21 to 7.x.

Re-roll

Removed the white space

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/5177237

I think there are times where the previous behavior might have been desired (e.g., depends on what you display on your site's access denied page), so I went ahead and mentioned this in CHANGELOG.txt. It's easy enough for someone to override it in their own submit handler if they need to.