Closed (works as designed)
Project:
Link checker
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 Mar 2009 at 06:43 UTC
Updated:
9 May 2009 at 11:09 UTC
delete link in "status/linkchecker" page
or
auto delete 404 links related node in a few day later
this great module set the 404 node unpublish , so i think is possible to set it auto delete.
if it's not a good idea to auto delete , i think it's better to add a delete link to the page "status/linkchecker" so that administrator can easily delete a node
or what's even better , add a check box like "admin/content/node"
Comments
Comment #1
hass commentedDo you really like to delete a node if only one link in a node is broken??? This is very risky - you may loose all your sites content! I'm not sure if this issue queue will fill up with such a feature... sounds too risky to me.
Comment #2
haojiang commentedhow about add a check box before the broken links?
then user will have the ability to choose.
the situation is that if some nodes is not "404" , but other code "500" and so on.
in "404" situation , the module will autoly set it "unpublish" , so it's ok for me ,because this node will not show up.
but if not "404" situlation, i stll want some nodes which have more than 3 error show to be set to unpublish.
so , is it possible?
but untill now , the module is enough for me .thx to this module ,i will have a lot time save.
Comment #3
haojiang commentedhow about add a check box before the broken links?
then user will have the ability to choose.
the situation is that if some nodes is not "404" , but other code "500" and so on.
in "404" situation , the module will autoly set it "unpublish" , so it's ok for me ,because this node will not show up.
but if not "404" situlation, i stll want some nodes which have more than 3 error show to be set to unpublish.
so , is it possible?
but untill now , the module is enough for me .thx to this module ,i will have a lot time save.
Comment #4
hass commentedWe cannot add a checkbox or delete link to the broken link as this is only a view of the broken links. It does not hold the links... they still exists in the content.
For future I think we need to add more error handling variants, but it's a bit tricky. For 301 and 404 we are pretty save, all other might need an users (technical) eye what is really broken as we cannot say for sure. This is the main issue here... unpublishing nodes with other error codes may come, but we need a different UI for this and need to be *very* carefully about the actions.
One thing I can think about is an option to remove a link from a content body, but I'm not yet sure how (seems impossible to be save). With
<a href="/foo">Example</a>this would end up inExample, but it can potentially destroy the context... often people writeClick <a href="/foo">here</a>that would becomeClick here... So I thought - it's better to review content manually... auto fixing is ultra difficult and will have fails positives.Comment #5
hass commentedChanging title to hopefully reflect the feature request. I'm setting to minor as I'm not planing to work on it as it's to risky for the content as written above. I nevertheless think we need to add a ignore feature in future as a site may have too many dead links and users will not fix this (for e.g. forums). On the other side maintainers are able to deselect forum nodes from link checks... so there may be no real to-do.
Automatically delete a node gets a WON'T FIX from me as we don't like to loose content.
Comment #6
hass commentedComment #7
hass commented"Unlinking" a link sounds too risky to me. I'm changing this case to by design. The ignore link feature will be implemented in #268946: Add ignore filter for links with buggy servers.