Closed drorsnir closed 2 years ago
I don't think the parser function approach is the right way, it allows editors to bypass the audits completely as well. Besides one needs to add it to every page/template.
Another extension shouldn't be required as this is a basic feature which should be provided by PageQuality itself.
I think the best way would be to specify the type of pages for a scorer in the settings page. It all depends on how we are able to classify such pages? Are they all in one namespace or do they all use a particular template?
Recognizing the type of page can be done in a few ways:
PageProps::getInstance()->getProperties( $this->getTitle(), 'ArticleType' );
Closing as per our discussion on Google Meet.
Not every audit is relevant to every article type in our system. For example, "emphasis_gov_symbol" isn't relevant to articles of type "guide". What do you think would be the best way to handle such exceptions? A parser function to set ignored audits, a hook that another extension will use to change the scorer list based on the article type, etc.?