Project

General

Profile

Feature #84990

Updated by Sybille Peters almost 6 years ago

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

 If linkvalidator is installed and broken links records are available, the broken external links could should 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