Posts: 68
Threads: 14
Joined: Sep 2015
BYOND Username: Xaviens
06-09-2016, 01:23 AM
(This post was last modified: 06-09-2016, 03:26 AM by Xaviens. Edited 1 time in total.
Edit Reason: Hansen
)
What it says on the tin.
This patch by ErikHanson no longer functions properly. When the AI examines a player, it gets a "Too far away" report.
To reproduce: Try to examine a player as AI.
Herp.
Posts: 215
Threads: 17
Joined: Jul 2015
Posts: 770
Threads: 157
Joined: Jul 2015
BYOND Username: ErikHanson
did someone summon me? I have literally no idea why it's failing, too bad i don't have commit access, it used to work just fine previously
Posts: 2,722
Threads: 143
Joined: Sep 2012
BYOND Username: Powmonkey
(06-09-2016, 05:04 AM)ErikHanson Wrote: did someone summon me? I have literally no idea why it's failing, too bad i don't have commit access, it used to work just fine previously
Actually, I didn't even know your patch was ever implemented because it seemed like it never got fixed.
I'm not sure your patch ever worked right.
I ran into a similar issue with my armor patch where I forgot to push a change I made on my local repo to the remote repo.
Zewaka had to manually change it once we figured out the issue.
Posts: 770
Threads: 157
Joined: Jul 2015
BYOND Username: ErikHanson
I'm like 99,99% certain it worked when the patch was posted, heck even the last revision went trough several local tests and all patch files are manually generated by me so it should work, unless someone else goofed it up.
Posts: 2,722
Threads: 143
Joined: Sep 2012
BYOND Username: Powmonkey
(06-09-2016, 08:33 AM)ErikHanson Wrote: I'm like 99,99% certain it worked when the patch was posted, heck even the last revision went trough several local tests and all patch files are manually generated by me so it should work, unless someone else goofed it up.
Well, either way, it seems to be broken now.
Posts: 2,039
Threads: 65
Joined: Nov 2014
BYOND Username: Zewaka
Character Name: Shitty Bill Jr.
06-09-2016, 10:57 AM
(This post was last modified: 06-09-2016, 11:42 AM by ZeWaka. Edited 1 time in total.
Edit Reason: im fix
)
Looks like this patch was not implemented correctly and fully. I'm looking into it currently.
EDIT: Fixed with the next push I make.