findologic / findologic-api

Library for sending requests to the Findologic API
MIT License
1 stars 3 forks source link

Split ParameterBuilder in two seperate classes #15

Closed TheKeymaster closed 5 years ago

TheKeymaster commented 5 years ago

At the moment the ParameterBuilder is taking care of config parameters and request parameters. Sine these are two entirely different things, let's separate them into their own classes.

TheKeymaster commented 5 years ago

EDIT:

The ParameterBuilder should actually be split-up again. There should be a JsonParameterBuilder and a XmlParameterBuilder. Maybe we should have three to ensure everything is clean. SearchParameterBuilder, NavigationParameterBuilder and SuggestionParameterBuilder. I think this should make everything more clear and easier to use.