search

joaofarias / csl-traffic

A WIP mod for Cities: Skylines to improve traffic.
90 stars 30 forks source link

Changed the way the transport tool is replaced, giving better compatibil... #3

Closed RosaryMala closed 9 years ago

RosaryMala commented 9 years ago

...ity with things. Particularly my mod that replaces NetTool

joaofarias commented 9 years ago

Ah, I see where it can lead to problems with your mod and I imagine there are others using your method. Thank you for showing this to me.

I will use your method and give you full credit but I'm not going to merge it for 2 reasons: 1- I've been changing the way I load things, including the TransportTool, and it will be easier for me to simply copy-paste the necessary stuff avoiding merge conflicts 2- I won't create the TransportToolReplacer and will instead put that behaviour in the Initializer class that is already up and running. Plus, if I ever need to replace another tool, I can simply add the functionality to the same method. There's no need to create a class for each type of tool for such a simple behaviour.

If I'm wrong on anything let me know. I'm seeing this on the phone so I may have missed something.

Thank you again for your work. It's greatly appreciated.