Optimize

[LeanderCS]

Summary by Bito

This pull request significantly enhances the Flask input filter module by optimizing data handling and validation processes. Key improvements include the introduction of new methods, refactoring for clarity, and the integration of a Cython-based validation mixin. These changes aim to boost performance, maintainability, and usability of the module.

[Copilot]

Pull Request Overview

This PR restructures core components by centralizing model classes, introduces Cython support for performance, and refactors validation and filtering logic into mixins.

• Migrated BaseValidator, BaseFilter, and BaseCondition into flask_inputfilter.models
• Extracted validation steps into ValidationMixin and data handling into DataMixin
• Updated default HTTP methods, import paths, and enabled dynamic Cython module discovery

Reviewed Changes

Copilot reviewed 193 out of 209 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
flask_inputfilter/input_filter.py	Switched to DataMixin.validate_with_conditions, updated default methods order
flask_inputfilter/mixins/validation_mixin/validation_mixin.py	Split filters/validators into two lists, updated signature
flask_inputfilter/helpers/parse_date.py	Replaced elif with if for type checks
flask_inputfilter/models/*	Moved all base classes (Filter, Validator, Condition) under models
setup.py	Dynamically discovers .pyx files for Cython extensions

Comments suppressed due to low confidence (4)

flask_inputfilter/mixins/validation_mixin/validation_mixin.py:19

• The docstring for apply_filters no longer reflects the two-filter-list signature. Update the description to mention both filters1 and filters2 parameters.

    ) -> Any:

flask_inputfilter/mixins/validation_mixin/validation_mixin.py:48

• The validate_field docstring still explains the old single validators parameter. Update it to document both validators1 and validators2 lists for clarity.

    ) -> Any:

flask_inputfilter/input_filter.py:50

• The default HTTP methods order has changed from the previous implementation and may no longer match the Cython make_default_methods() output. Consider aligning this list or delegating to make_default_methods() to ensure consistent behavior.

        ]

flask_inputfilter/helpers/parse_date.py:18

• [nitpick] The two consecutive if statements are mutually exclusive checks. It may improve readability to use elif for the second and third type checks so it’s clear only one branch applies.

    if isinstance(value, date):

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud