It will be nice if instead of Node we have this functionality for all entities.

Comments

gaurav.kapoor created an issue. See original summary.

devasghar’s picture

devasghar
He
Primary language Pashto
Location Peshawar
commented

Sure, nice idea. I think i will have some time next week. I'll work on it.

akashkumar07’s picture

akashkumar07 commented
Status: Active » Needs review

Hi @dev.asghar,

I have modified your module to support all entities. Please find the updated module in the below repository url .
https://bitbucket.org/AkashKumarOSL/entity_delete_redirect
Note:- I have not included 'user' entity yet.

Please review.

devasghar’s picture

devasghar
He
Primary language Pashto
Location Peshawar
commented

Sure Akash, do you rekon you could add User entity as well. I'll have time to review this next week. Too much on the plate at the moment.

akashkumar07’s picture

akashkumar07 commented

Yes, we can add User entity as well. But, I am facing some issue with 'user entity redirect '.
So, I have hided the User entity from form.

Thanks @dev.asghar

devasghar’s picture

devasghar
He
Primary language Pashto
Location Peshawar
commented

Ok, i'll have a look at that as well.

Appreciate your time on this.

seanr’s picture

seanr
he/him
commented
Version: 8.x-1.x-dev » 8.x-3.3
Status: Needs review » Needs work

Will need to take a deeper look at this, but I would suggest creating an issue fork off the current dev branch - quite a lot may have changed since this was created (including D10 support and drop of D8). I think this is probably something we'll want to do for 4.x as it will potentially involve renaming the module.

Additionally, a cursory glance at the code in the bitbucket repo revealed a number of code style issues including some indentation stuff and dpm() functions peppered throughout.