Project

General

Profile

Feature #84990

Updated by Sybille Peters over 6 years ago

Currently, RteHtmlParser::markBrokenLinks checks only for broken *page links* and marks them accordingly. 

 If linkvalidator is installed and broken links records are available, the broken *external links* could be marked in the same fashion.  

 Also *file links* and other links could be considered.  

 This would not only make fixing broken links using the linkvalidator much easier, but would also make this information available whenever a record is edited.  

 Based on some issues Christian Kuhn raised in the patch, the following should be be considered: 

 * having dependencies from core to linkvalidator would be bad practice. Thus, the corresponding code should be moved from RteHtmlParser::markBrokenLinks to a place, such as in core/Classes/LinkHandling  
 and a hook or something similar should be added, which can be used by Linkvalidator  
 * must add a check if in backend context? 

 Also, setting the style, must be done differently, as this currently does not work with ckeditor, but this is being addressed by #84987 ("Add css style for broken page links in ckeditor"). 

Back