LoneGazebo / Community-Patch-DLL

Community Patch for Civilization V - Brave New World
Other
282 stars 157 forks source link

CityStateDiploPopup.lua #2328

Closed Iamblichos closed 7 years ago

Iamblichos commented 7 years ago

check typo on line 725 reads:

if (minorPlayer:IsMinorCivDisplayedQuestForPlayer(activePlayerID, MinorCivQuestTypes.MINOR_CIV_QUEST_UNIT_GET_CITY)) then

should be:

if (pMinor:IsMinorCivDisplayedQuestForPlayer(activePlayerID, MinorCivQuestTypes.MINOR_CIV_QUEST_UNIT_GET_CITY)) then

this is in NON-EUI compatibility files...not sure about EUI...might want to check that one too.

LoneGazebo commented 7 years ago

EUI uses minorPlayer, no-EUI uses pMinor. Because bc1 felt he had to change everything for fun. :/

I'll fix.

Iamblichos commented 7 years ago

I'm telling ya we should just make our own version (what we need and want) so we don't have everything going through that dumb core library. That thing is a hog.

LoneGazebo commented 7 years ago

Its something for sure. But boy golly do I hate interface work. Also bc1's made it clear that he does not want mod mods of EUI (we had quite the fight about my compatibility files being an 'infringement' on his code).

Iamblichos commented 7 years ago

Yeah but he doesn't have a monopoly on making UI interfaces for civ 5....I mean scrap the whole EUI. we don't have to copy his work. Just make our own(what we need).

Iamblichos commented 7 years ago

The other advantage of doing this is getting completely rid of the EUI and non-EUI versions.

LoneGazebo commented 7 years ago

We could. But people like EUI. A lot. And that's hard to break. People are stubborn, and most would still want EUI. :)

Iamblichos commented 7 years ago

Then we give them what they want without being EUI and without using any of his code. You know that thing is a POS.

Iamblichos commented 7 years ago

Btw- I still get late game crashes from it that break on the EXE and the UI thread.

LoneGazebo commented 7 years ago

Yeah. We could do an exploratory post asking users what the 'most essential' aspects of EUI are. Go from there.

LoneGazebo commented 7 years ago

EXE and the UI thread?

LoneGazebo commented 7 years ago

We could make an exploratory post regarding the 'essential' features of EUI, see what users really or feel necessary.

Iamblichos commented 7 years ago

Yes

LoneGazebo commented 7 years ago

Yes to what? Also, mind re-linking the crash? I haven't had a late-game crash in forever.

Iamblichos commented 7 years ago

Yes to making an exploratory post.

LoneGazebo commented 7 years ago

Gotcha.

Iamblichos commented 7 years ago

Yeah I don't have crash now but it's from actually playing the game and hovering over a city banner or picking a unit etc. usually involves a lot of data on the screen.

Iamblichos commented 7 years ago

Next time I have it again I'll let you know.