Change tags to on operations to group methods differently about xero-openapi HOT 2 OPEN

I love the idea of injecting just a ContactApi etc.

As a potential solution that could satisfy both camps, could the API also generate an AccountingApi class that just had methods to return an instance of each of the apis?

Then you can choose to inject the individual apis, or inject the AccountingApi and call $accountingApi->contacts()->getContacts(....) etc? With the contacts method simply returning an instance of the ContactApi created with the Guzzle client and Xero Configuration class passed to the AccountingApi constructor?

I don't think we should group by scope as it then becomes a bit confusing to work out what we need to inject to access what.

from xero-openapi.

Each endpoint can have multiple tags, so accounting could group everything in the Accounting API, and then additional tags could group other collections of endpoints. I could imagine there would be some overlap. Being able to run a small client with a collection of related enpoints to perform a task that needs just those enpoints, I'm sure will be useful. Having a single client to span the full range of endpoints is also very useful for tasks that need it (e.g. we query bank accounts, transactions, invoices and clients all in one job - these things are all very closely related.

So I agree with @hailwood - a tag for everything, and tags for groups of related functions. When generating code, you can always choose one or the other (or both) by generating against chosen tags only.

The way the PHP clients are built, I think is very close to how the clients for other languages would be built, at least when they are generated through the https://github.com/OpenAPITools/openapi-generator tool. All the languages are given the same data structure to work from, and that colours the structure of the code that is generated.

The PHP code is quite long (an understatement) because of the way the templates build it "long-hand" with very few shared classes. Two points to note here: firstly PHP doesn't care much abut the length of these classes. Secondly it can be shortened somewhat with custom templates. The OpenAPI builder v5 makes it even easier to add your own custom support/utility classes in the generated code to reduce the size even more.

Just throwing my 2p into the ring.

from xero-openapi.