Project

General

Profile

Task #92557

Updated by Sybille Peters over 3 years ago

Clean up code: 

 * Use strict in all classes 
 * Add type hints 
 * checkLink third argument LinkAnalyzer: this is only used in Linktype "LinkHandler" and is used to pass TSconfig. The lowlevel class / function inherit from AbstractLinktype should propably not depend on LinkAnalyzer. It might be possible to pass the configuration differently, i.e. this is already done in ExternalLinkType::setAdditionalConfig 
 * possibly move all queries to the repository class BrokenLinkRepository 
 * LinkAnalyzer::init(): second parameter: $pidList: "string|array $pidList List of comma separated page uids in which to search for links, can be an array too": error-prone, use array error-prone

Back