5 commits

Author	SHA1	Message	Date
Daimona Eaytoy	6e27a9ddb3	Cleanup variables-related classes ... Change-Id: I20a7fe1a40255043ed0d125dee61ea6052dda69c	2021-01-02 18:19:38 +01:00
Daimona Eaytoy	762d71c51d	Create a dedicated namespace for variables-related classes ... Some cleanup is left for later to keep the diff easier to read. Change-Id: Ife445b5e47e707ab77ec867ac3b005866aa74ef2	2021-01-02 18:16:48 +01:00
Daimona Eaytoy	d3b330b6d4	Create a VariablesManager service ... This makes VariableHolder a true value object, and introduces a stateless service, VariableManager, to operate on it. Note, in theory, this new service is still cyclically coupled with LazyVariableComputed. However, it's now two stateless service being coupled, not two smart/god value objects, so we've still earned something. For now, the dependency is hidden by using a callback. Some alternatives for that are mentioned in a code comment. Bug: T261069 Change-Id: I2f2c84c8e91472ba36084a8bbb4a923f6e04354b	2021-01-02 17:15:31 +00:00
Daimona Eaytoy	3e0c30ff92	Allow the parsers to return extra info ... This is achieved by creating a new ParserStatus class. Aside from the result of parse(), it contains whether the cache was warm. This can be used to differentiate profiling data as part of T231112. Another use case is returning non-fatal warnings (T269770). Change-Id: Ifcbda861ce1a44bbe9bffba5b83cd9ef338a8dba	2020-12-11 15:03:23 +00:00
Daimona Eaytoy	da1c71ec4c	Move parser classes to a dedicated namespace ... Names were kept for now. Change-Id: Ib2eb5d7b523a64f2a0f72fdcdde2043a76cc9a37	2020-12-09 01:30:20 +00:00

Renamed from includes/parser/AbuseFilterCachingParser.php (Browse further)