GG Refactor

[Erkelinux]

Refaktorerte et par småting når jeg hadde noen minutter til overs forrige uke. Viktigst er deprecation av alt som har med isGenerated å gjøre, det skal forhåpentligvis bli helt borte veldig snart.

Samtidig vil jeg endre begrepsbruken til å ikke handle om "resolvers", men hva det burde hete kan gjerne diskuteres mer. Her foreslår jeg å bytte isResolver, som kom av en svak tidlig forståelse av Kickstart-resolvere, til createsDataFetcher.

[andreahn]

Skal vel ikke være datafetchers i flertall for ett felt? 🤔

Suggested change

	public boolean createsDataFetchers() {
	public boolean createsDataFetcher() {

[Erkelinux]

Tja, jeg ville at det skulle gå opp med det eksisterende interfacet, som er felles på tvers av fields og types. En type kan produsere flere datafetchers, og derfor kalte jeg bare alt createsDataFetchers. Derfor ønsker jeg innspill til hva annet det kan hete for å gi mening overalt

[andreahn]

Okei, så da har dette en forskjellig betydning avhenig av om det er et felt eller et "objekt" som inneholder felt? For felt, true hvis det trenger en data fetcher generert, og true for typer hvis den har et felt somt trenger data fetcher?

Hva med å skille på de to?

createsDataFetcher() som kan være true for felt, men aldri for en type

Og for typer osv: createsDataFetcherForChildren() eller noe i den duren? 🤔

[Erkelinux]

Ja, det er ca. det det betyr. Tidligere helt alt dette bare isResolver.

Vi kan skille på dem, men interfacet GenerationTarget blir litt weird.
Jeg kan sette det opp på den måten for nå, men vil nok rydde litt opp her i en større runde etterpå.

[andreahn]

Jeg syns dette skurrer litt fortsatt, men trenger ikke å vente med å merge for min del - i hvert fall hvis du har flere endringer i tanken 😄