Logistic Request Manager (continued)

by daeruun

Allows you to save and switch between custom logistic request presets, as well as request blueprints. Optionally presets can be pushed to constant combinators to use the circuit network to manipulate requests.

Utilities
3 months ago
0.18 - 1.1
24.6K
Logistic network

b [1.1.24] Space Exploration: Crash opening LRM when viewing SE Signal Reciever UI [fixed]

1 year, 6 months ago

Reproduction steps:
Place a Space Exploration Signal Receiver
Open it's UI by clicking the signal receiver
Open Logistic Request Manager UI via button, crash occurs

Error MainLoop.cpp:1284: Exception at tick 26942572: The mod Logistic Request Manager (continued) (1.1.24) caused a non-recoverable error.
Please report this error to the mod author.

Error while running event LogisticRequestManager::on_gui_click (ID 1)
LogisticRequestManager/scripts/blueprint-requests.lua:19: attempt to call field 'get_logistic_point' (a nil value)
stack traceback:
LogisticRequestManager/scripts/blueprint-requests.lua:19: in function 'get_inventory_entity'
LogisticRequestManager/gui.lua:435: in function 'set_gui_elements_enabled'
LogisticRequestManager/control.lua:650: in function 'close_or_toggle'
LogisticRequestManager/control.lua:64: in function <LogisticRequestManager/control.lua:48>

Also occurs if a research finishes while the signal receiver UI is open

1 year, 6 months ago

Hi!
Thanks for the report. I will look into it!

Daeruun

1 year, 6 months ago

Fixed in 1.1.25.

Daeruun

1 year, 6 months ago
(updated 1 year, 6 months ago)

https://github.com/Daeruun/LogisticRequestManager/issues/11

The PR attached to this fixes issues potentially when research finishes while the signal receiver UI is open, as the 2nd issue, which was still occurring with 1.1.25.

It is currently a draft PR because I have not tested it thoroughly yet but there does not appear to be much to test for breaking changes

1 year, 6 months ago

That is great. I oversaw the 2nd issue when I pushed the last version.
May I mention you in the log?
I'll do some testing tonight and then push it to the portal.

1 year, 6 months ago

Yeah thats fine. Go ahead and use my username here instead of the one on github if you are going to. Thanks!

1 year, 6 months ago

Thanks again.
I did some more testing and just pushed the updated package to the portal.

New response