Jump to content

Adding compatibility between two sorting mods


FriedStephen

Recommended Posts

On your first two items:

1. For this purpose, I would keep the quest running as you suggest, as it seems to work that way and rewriting it doesn't seem necessary, but there are other ways that wouldn't require a quest script, just to keep you fully informed. I had no idea the label maker wasn't linked to that. Again, I didn't really even know the majority of what the mod was intended to do, so I just redid the code this way and more or less learnt by gleaning whatever I did as that happened.

2. When you use If and a variable, with no operator, it's a more efficiently processed instruction to check is the variable 'true', where 'true' is any value that isn't 0. The performance you gain is negligible, really, but there's no reason not to use it and these negligible gains do add up to something more obvious if enough of them are utilised (several hefty ForEach loops with a particularly fast processing time might be a good example).

 

You may have messed up something, yeah, but I didn't even know what a lot of the code was intended to do until after I'd been over it a few times, so it makes sense that the error was on my end. Though it is odd that you're running to problems I never did. You are starting from a new game I hope? or at least a save that didn't have any form of this mod installed when the save was made?

 

Still, combining whatever you like from my updated code with what works on your old version sounds like a good plan to me.

Link to comment
Share on other sites

  • Replies 42
  • Created
  • Last Reply

Top Posters In This Topic

Well, this should be my final update on the plugin. I've done what I said, and merged both our previous versions. I also made a few tweaks and applied some polish. Most notably, I added a "Crafting Parts" label which is only available if Universal Item Sorter is loaded. Also, instead of messing around with "dumping everything but equipped items" with the mobile auto-sorter, I compromised and simply added the option to directly access the sorting container from the mobile auto-sorter; so it's still not very functional as a cheat-y item, but it's a little more versatile than before, just as I intended.

 

All I need to do now is write up some documentation, then release the mod. Once again, thanks for everything you've done, and all due credit will definitely be given.

Link to comment
Share on other sites

  • Recently Browsing   0 members

    • No registered users viewing this page.

×
×
  • Create New...