When you select "Edit Content" and then delete the node, the page redirects to the front page rather then back to the page that references it. Is there any way to make it so that it returns to the parent page like when you save it?

CommentFileSizeAuthor
#6 2039093-1.patch2.58 KBmanojapare
#4 2039093.patch2.74 KBmanojapare

Comments

bryancasler’s picture

Version: 7.x-1.0-rc1 » 7.x-1.x-dev

Also experiencing this issue in the latest dev release.

Micha1111’s picture

Version: 7.x-1.x-dev » 7.x-1.0-rc4
Component: Feature Request » Code
Category: Feature request » Bug report
Issue summary: View changes

This issue is 2 years old, but the problem still exists in rc4.
Marking this as a bug, because the funcionality is broken.
What happens with the data in entityreference field of the parent node ? Nothing is selected, if you open the parent form again !

BTW, it is very dangerous to allow users to delete such referenced nodes, because they can already be selected in other parent nodes. This ends up in unexpected data loss.

jygastaud’s picture

Deleting content is a core function and it is not the responsability of Entity Connect to deal with deleted nodes.
Entity Reference should.

I agree that it would be good to still come back to the form but it's not that simple as it can look.
Maybe we can remove the delete button when when the forms is load via entity connect button.

We need to check pro and cons of altering "normal" functionnality.

manojapare’s picture

Status: Active » Needs review
StatusFileSize
new2.74 KB

Please find patch file for the issue. In this patch, I have carried over the build cache id to node delete confirm form and thus entityconnect module can redirect back to parent form using the same.

Status: Needs review » Needs work

The last submitted patch, 4: 2039093.patch, failed testing.

manojapare’s picture

Status: Needs work » Needs review
StatusFileSize
new2.58 KB

I have corrected the patch path and attached it.

rakesh.gectcr’s picture

Status: Needs review » Reviewed & tested by the community

The patch looks good. Passing the $cid, is great work. I am moving this to RTBC

  • jygastaud committed 4398d94 on 7.x-1.x authored by manojapare
    Issue #2039093 by manojapare: Deleting content should return to parent...

  • jygastaud committed 263b8d7 on 7.x-2.x authored by manojapare
    Issue #2039093 by manojapare: Deleting content should return to parent...
jygastaud’s picture

Version: 7.x-1.0-rc4 » 7.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Thanks @manojapare for the patch.
I just push it in 7.x-1.x and 7.x-2.x branch.

Status: Fixed » Closed (fixed)

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