Skip to content

Conversation

@AteZenmo
Copy link
Collaborator

@AteZenmo AteZenmo commented Feb 3, 2026

Also J_EAConversion has 2 constructors now, one with asset flow catagory and one without.

J_EAConversion heeft nu een private construct functie die de functionaliteiten van de oude constructor bevat.
afbeelding

De oude constructor (plaatje de onderste) roept die gewoon aan en meer niet. De extra constructor met AssetFlowCatagory doet dat ook, maar zet eerst ook nog eens de asset flow catagory. Deze private construct functie is nodig, omdat je niet weg komt door this('constructor attributen') te doen. Want dan wordt registerEA aangeroepen voor de asset flow catagories gezet zijn, en dat gaat fout (this calls moeten namelijk eerste zijn wat er gebeurt in een constructor). Ook kan je niet de kleine de 'grotere' constructor laten aanroepen zoals we eerder wel eens hebben gedaan, omdat je een Option list argument van een constructor niet mag vullen met null.

Dit is dus de enige manier denk ik om het netjes te doen.

Verder heb ik dus dieselGenerator en MethaneGenerator toegevoegd aan OL_EAtype en AssetFlowCatagory. Ook aan de resultsUI in pull request: Zenmo/zero_results_UI#102

Ik heb het getest, het werkt prima, en er kan ook niet zoveel mis gaan.

-> Mochten jullie diesel willen veranderen in brandstof ofzo, kan dat natuurlijk, ik twijfel daar ook wat over, maar ik dacht diesel kan wel, want het is eigenlijk altijd een diesel generator. Feedback is welkom.

Also J_EAConversion has 2 constructors now, one with asset flow catagory and one without.
@AteZenmo AteZenmo merged commit 9ff3b0c into main Feb 4, 2026
@AteZenmo AteZenmo deleted the diesel-methane-generator-conversion-option-additions branch February 4, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants