Slaves not being removed from DB (I am guessing - in presence of Imperial slaves) about trade-dangerous HOT 5 CLOSED

Yay. I'll look into it.

from trade-dangerous.

So, the problem is, the plugin doesn't remove no longer existing items from the DB. I just hadn't implemented it.

I /think/ I have a solution, but I haven't tested it yet and I don't really have the time to do so right now.

I'm uploading it to the dev branch now, in case you want to test it. I'll push it to main once we know for sure it works.

from trade-dangerous.

Other than that odd one I found, it's likely somewhat difficult to find test cases (or we would have found this before).

Is it reasonable to add "bogus" commodities to a given station by adding fake, but valid lines to the DB and then see if the plugin removes it?

from trade-dangerous.

Do an update, remove some entries from one or more of the stations in the source file and save it. Do a second update with the force option, see if the entries you removed from the source file are gone in the DB.

The code I added, assuming no bugs or unintended results, does the following:

Set current station to -1, create empty dict.

For each item in the source file:

Check if current station matches item's station. (Will be false for the first item.)

If not, delete all items in the current dict set to false, delete the dict, set current station to the new station, and create a dict of all the possible items set to false

If so, set the current item to true in the dict.

from trade-dangerous.

Fixed in 8a43404

from trade-dangerous.