-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More farming issues now with Plant Easily bricked by Xportal #54
Comments
The above steps are unnecessary, simply place more than 1 piece at a time with PlantEasily (which happens to happen during auto crop replanting as mentioned by the OP) and watch your console light up. I would imagine this happens with similar mods (MassFarming, Farming) as well but haven't confirmed.
Followed by the always repeating, even while game is paused or after quitting to title screen:
I would also add that these pieces do not have wear & tear components. Edit: Issue occurs even when placing a single piece with the cultivator while PlantEasily is installed, even when PlantEasily's features are disabled via its config (causing all of its patches to return before any code is executed). |
Judging by the comments in your code, you're clearly doing some strange things to work around some niche issues. Despite this, here's a couple things I noticed that are suspect to me. XPortal.Patches.Piece.CheckWearNTearCreationTime(bool) is seemingly never called with a false argument for the delayed parameter (unless I missed it elsewhere), does it not always queue itself to be run again without ever reaching the latter portion of the method? Your Piece.SetCreator Postfix patch doesn't check if m_WearNTear is null or not, nor is it checked anywhere else that it's referenced. I would suggest at least 1 initial null check before proceeding with your other code.
|
That is correct @AdvizeGH, please see here for reference as to why: I will look at this issue as soon as I have time. |
That was a really interesting read. This issue recently cropped up elsewhere but it was assumed to be limited to Transpilers. I can't think of a workaround that I could offer except to explore alternative entry points... perhaps TeleportWorld.Awake() or somewhere in there. |
Hi @Astral-Crescent, thank you very much for reporting this issue. I have just released v1.2.12 which fixes this problem. |
I am mentally handicapped and cannot fill out your form the way you want.
Please listen to me, anyway.
Copy-Pasted from my bug report on Plant Easily now that I know it is Xportal's fault.
"Since the recent update, When I try to mass harvest it only harvests 1 node but yields Many crops, causing unfair duplications, and then the multiple crops are all replanted ontop of each other, leaving them unable to grow, and it consumes no seeds, and remains as a fully ripe crop.
edit: This is being triggered by another mod called XPortal, for... some reason. If you can sus out why that might be, I would appreciate it. "
When I attempt to do the above with the mod Plant Easily, I get an infinite readout of errors to do with Xportal.
To replicate: Install Plant Easily by Advize, enable mass harvest and replant when harvested (not sure if needed, but these are the features I was bug testing when I noticed this error with Xportal), attempt to mass harvest or harvest crops.
The text was updated successfully, but these errors were encountered: