Crowfunder / Crowfunder/Cogmaster

Config Parser Fixes

Open
#1 0 comments 0 reactions 0 assignees View on GitHub
BackEnd bug help wanted
Dominant language
Java
Stars
4
Forks
0
Avg merge
22m
Merged PRs (30d)
1

Description

There are several minor issues with the parser.

## Lists have redundant nesting.

![Image](https://github.com/user-attachments/assets/fa050c36-3b7a-4042-a535-be996aa8f2e5)

![Image](https://github.com/user-attachments/assets/6836ffd9-ba7e-4311-8869-782c3cbb3b22)

Despite the fact that a list is a parameter value, the parser detects first detects it as a nested parameter (created as a hashMap)

**Intended:** If a parameter's value is a list, it should not be treated as a nested ParameterArray.

## Parsing Bugs

There are still numerous bugs lurking within the parser. Here are two examples found in `Brandish` config.

![Image](https://github.com/user-attachments/assets/ccdfc181-af85-403b-97d2-32a1d145d547)

![Image](https://github.com/user-attachments/assets/73124ce0-1d19-4f64-b2b8-76f7d091fb7e)

## Refactoring
The code is in **dire** need of proper refactoring.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the configuration parser and reproducing the examples from the Brandish config mentioned in the issue. Investigate why list-valued parameters become nested ParameterArray or hashMap structures. Done means list values parse without redundant nesting and the reported parsing examples produce the intended results; the requested refactoring scope is not defined.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.