BGforgeNet / FO2tweaks

Fallout 2 tweaks
https://forums.bgforge.net/viewforum.php?f=26
122 stars 11 forks source link

Crash when using Armor inventory filter in Barter screen. #2

Closed TallgeeseHeaven closed 7 years ago

TallgeeseHeaven commented 7 years ago

What it says on the tin. All other categories work fine. Armor also filters fine on the standard inv screen.

TallgeeseHeaven commented 7 years ago

Ah, this actually appears to be an issue with sfall 3.8.3. If I make it 3.8.2 this does not happen.

EDIT: It is also fixed in 4.0.

burner1024 commented 7 years ago

Thank you for report and testing with various versions, I can confirm it. Unfortinately, Molotov cocktail component requires 3.8.3 to function properly, and 4.0 is not out yet. When it is, I'll update fo2tweaks.

TallgeeseHeaven commented 7 years ago

There's actually a bigger problem: sfall 3.8.3 also breaks turning in certain quests, like Westin's holodisk to Lynette. So people using fo2tweaks might attribute that to your mods.

You could package a 4.0 ddraw.dll.

burner1024 commented 7 years ago

That is annyoing. 4.0 is not released yet, is it? I might rather package 3.8.2 and disable molotov component for the time being.

TallgeeseHeaven commented 7 years ago

Nah, not fully released, but there is a dll for it in the issue tracker, like here:

https://github.com/phobos2077/sfall/issues/94

NovaRain commented 7 years ago

Actually it's not using "armor" inventory filter caused the crash, any category of items that has less than 3 items will crash upon clicking that category. (e.g. if you only have a spear, using weapon filter in barter screen will also crash the game.) If I commented out the two lines with "call Add_ReservItem" the game won't crash, but the filtered inventory display will be glitched for the category with less than 3 items, until you try to pick up items.

EDIT: got it, the crash is related to ItemCountFix added in 3.8.3. I probably messed up something when backporting it from 4.0.

burner1024 commented 7 years ago

Looks like it will be a while until 4.0 is production ready. I've downgraded to 3.8.2 and released it as 7.1, disabling Molotov component. Will get back to this later.

FakelsHub commented 7 years ago

crash is related to ItemCountFix added in 3.8.3.

@NovaRain, What kind of fix is ​​this? What was wrong with counting items...

NovaRain commented 7 years ago

@FakelsHub

Added a fix for the bag/backpack exploit that lets you keep items that are supposed to be removed from inventory

The same fix in 4.0 works fine, just the backported one in 3.8.3 is causing the glitch.

phobos2077 commented 7 years ago

Guys can you check debug.log when running this script, if there is any error messages when using 4.0 and when using 3.8.3?

burner1024 commented 7 years ago

None in 3.8.3, it seems, the log it attached. debug_3.8.3.log.txt

Crash message below.

---------------------------
Application Error: C:\games\fallout2\FALLOUT2.EXE
---------------------------
The instruction at 00471bda referenced memory at 0000002d
The memory could not be read from

Click on OK to terminate the application
---------------------------
OK   
---------------------------
NovaRain commented 7 years ago

3.8.3 caused the game crash when calling destroy_object or destroy_mult_objs because the way VS2008/2010 (for compiling 3.8.x) handling some code is different from VS2015 (for compiling 4.0). I've fixed the bug in the upcoming 3.8.4.

burner1024 commented 7 years ago

Seems 3.8.4 is allright, but I would also like to wait for https://github.com/phobos2077/sfall/issues/98 being fixed, otherwise installation fails on EcCo.

burner1024 commented 7 years ago

Looks like it's going to be a long wait. Maybe I should just go with 3.8.4 for now.

burner1024 commented 7 years ago

fixed in v8

FakelsHub commented 6 years ago

Обнови скрип фильта, он у тебя еще ранних версий в котором есть критические баги.

burner1024 commented 6 years ago

какие

FakelsHub commented 6 years ago

Если в инветаре лежит свыше 99999 монет, то они будут превращаться в 500 :-) Баг с характеристиками брони, обнуляющий их.

burner1024 commented 6 years ago

Завел баг, как руки дойдут посмотрю.

burner1024 commented 6 years ago

@FakelsHub Потестировал новую версию, какая-то лажа получается. У половины партийцев есть фильтр, у другой нет. Например, есть у Кэссиди, нет у Майрона. Выгоняю Кэссиди, беру его обратно - теперь у Майрона фильтр есть, у Кэссиди нет. Это с 4.0.2 и AllowUnsafeScripting=1.

FakelsHub commented 6 years ago

Дай сохранку с партийцами. Обмениваешься с партийцами как - в бартере, или через кражу?

burner1024 commented 6 years ago

Через кражу. slot22.zip

FakelsHub commented 6 years ago

сохранку забрал - это RP или ваниль?

burner1024 commented 6 years ago

killap's patch

FakelsHub commented 6 years ago

Ну да есть такой косячек, только это происходит когда партиец в движении, а если обмениваться со стоящими на место то все нормально. Это кстати всегда так было, механика скрипта такая (чузен должен смотреть лицом на партийца, и расстояние между ними должно быть не больше 1-го гекса, только тогда срабатывают панельки), и ты видать только сейчас заметил :) Попробую исправить этот недочет в механике.

burner1024 commented 6 years ago

Да нет, не могу сказать, что только в движении. Четко у Маркуса и Майрона нет панелек, при любом раскладе.

FakelsHub commented 6 years ago

Ну незнаю, у меня все есть) я на 1С сверху накотил патч киллапа и все. Версию скрипта надеюсь Advanced 1.3.5 используешь?

FakelsHub commented 6 years ago

Проверю потом как стандартная версия работает.

burner1024 commented 6 years ago

да Advanced