CMU-17313Q / NodeBB

GNU General Public License v3.0
0 stars 45 forks source link

Refactoring code in src/controllers/mods.js #68

Open mayTkhin24 opened 2 weeks ago

mayTkhin24 commented 2 weeks ago

Resolves #39

I refactored code in src/controllers/mods.js to reduce cognitive complexity which was 20.

Several helper functions were added to make the code easier to read and simpler to debug. The helper functions are as follows:

initialize(req): Sets up needed data. checkAccess(...): Checks user permissions and updates local variables. parseFilters(req, filters): Processes filter parameters from the request. adjustFiltersForMod(filters, res): Updates filters based on moderation. determineHasFilter(filters): Checks if filters are applied. parseSort(req, sorts): Handles sorting parameters. fetchData(req, filters, sort): Retrieves the required data. getSelectedUsers(filters): Gets user details for selected filters.

coveralls commented 2 weeks ago

Pull Request Test Coverage Report for Build 10720589831

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/controllers/mods.js 25 26 96.15%
<!-- Total: 25 26 96.15% -->
Files with Coverage Reduction New Missed Lines %
src/controllers/mods.js 2 76.03%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 10550029021: 0.02%
Covered Lines: 22336
Relevant Lines: 25592

💛 - Coveralls
sonarcloud[bot] commented 1 week ago

Quality Gate Passed 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 SonarCloud