SamuriHL
UHD Guru
- Joined
- Jan 28, 2007
- Messages
- 15,048
- Likes
- 2,553
nception has been re-assigned to you.
(FYI - the re-assign function is brand new code I had to add in to accommodate an earlier situation.
This was its second use.
Happily seems to be working great - :=))
Nice.
Awesome.
Worked ok to get a report for Doom under the normal use case.
That's a plus. lol
I notice now that if there is no Forum Name everything is locked down.
That's great!
It was always supposed to do that. And did in my testing but there were some weird scenarios where it didn't work as planned.
Even found if I clear my name and re-boot the Scanner the Forum Name re-populates itself.
Very nice!
Looks good so far with these few quick preliminary tests.
Right, it'll only update the storage of your name if it's a valid change and only once you leave that field. I.E. it won't update anything while you're typing it, but once you click out of that field, if it's valid, it'll update it. If it's not valid, i.e. you erase it, it won't update the stored value. This, too, was how it was always meant to work and did in my testing prior to this release. However, as with everything else, I found some nasty bugs in that code so it probably didn't work nearly as well as was intended. It will now.
Certainly.
When its time, let's make sure to co-ordinate so both sides have that in place.
Yea, that's fine. Likely in the next few days unless we see some kind of showstopping issue. There could still be some whacky edge cases kicking around. These changes I made initially introduced a couple new bugs that I quickly squashed before releasing it, found in my own testing with the report I made earlier. All those modes made the code far less clean than I would have liked and that's what I think caused the issues we were seeing with blank names in reports. It shouldn't happen, but there were a lot of different ways to enable things in the scanner prior to these fixes that should not have been allowed. Now everything goes through one entry point for enabling/disabling scanner features. So we'll see and if it's good we'll set it to required so we can nuke those blank name reports.