I administer over a dozen Drupal sites. Each has a non-technical user that is responsible for day-to-day administration and content creation. My goal for these functional improvements to the customerror module is to make modest enhancements that are accessible to the non-technical user. All of these are possible without the customerror module with a dozen lines of PHP, however that is not an option for many site maintainers. So making these enhancements "turnkey" options is my goal.

I also have some security questions/concerns that I want tackled, but first things first...

So the enhancements I have in mind are:

  • For 404 errors
    • If search module enabled
      • Option to redirect user to search results page if anything contextual in url
      • Option to add link to search results page if anything contextual in url
    • Checkbox to add link to home page - perhaps should just be link to "other" page so that admins can select either home page or site index. I don't like this option in general, but it's what many of my constituents mention.
    • Move Redirect List to 404 fieldset since it has nothing to do with 403 or any other errors.
  • For 403 errors
    • Checkbox to enable message appended to page indicating that:
      • if not logged in, logging in might provide access to the page, but might not, along with a login link to /user?destination=xxx.
      • If logged in, they just don't have access to page.

I understand that none of these are difficult to do with Php, but nothing this module does is difficult in core to do for someone that knows Drupal & Php.

Please any interested users, share your feedback.

Comments

scafmac’s picture

scafmac commented

Ha ha - nix the Redirect List move. It's smart enough to work for 403s & 404s. My mistake.

kbahey’s picture

kbahey commented
Status: Active » Closed (won't fix)

* For 404 errors
o If search module enabled
+ Option to redirect user to search results page if anything contextual in url
+ Option to add link to search results page if anything contextual in url

There is already a module for this, check it here http://drupal.org/project/search404 no need for duplication.

o Checkbox to add link to home page - perhaps should just be link to "other" page so that admins can select either home page or site index. I don't like this option in general, but it's what many of my constituents mention.

Why do you need to do this, if there is free HTML and a link to anything can be done? This is making things too narrow, while a more generic way is already available.

* For 403 errors
o Checkbox to enable message appended to page indicating that:
+ if not logged in, logging in might provide access to the page, but might not, along with a login link to /user?destination=xxx.
+ If logged in, they just don't have access to page.

You already have PHP that you can paste in. Check for whatever you want and make it emit the text you want.

I don't see that any of the above is really needed, since they are achieved by other means that are quite easy.

scafmac’s picture

scafmac commented

You're right about the 404s - I missed the other module.

As far as the other stuff - You've made an insecure, experts-oriented module that is inaccessible to many folks - since I thought I was going to be using it for the 404 errors, it seemed like a good idea to contribute some code to make it a decent module, maybe even one that had a nice UI and saved people time. But hey, it's your baby, you do what you will.

kbahey’s picture

kbahey commented

Comments like these are not helpful at all:

You've made an insecure, experts-oriented module that is inaccessible to many folks

it seemed like a good idea to contribute some code to make it a decent module, maybe even one that had a nice UI and saved people time.

But hey, it's your baby, you do what you will.

Being abrasive will not get things forward for anyone ...

I have given reasons for not accepting these features. These will only cause featuritis and code bloat.