Rather than "unfix" the master branch, I added a provisional fix to the fix and wrapped the code we moved in an "if local" conditional, which I think was the problem with the original patch.
We can test this for expected behavior now.
Scott Cantor November 16, 2017 at 9:42 PM
Without digging in deeply, I think the problem is that the logic we moved was inside a conditional that applied only to the m_local == true case (a local file) and we moved it to a spot not guarded by that condition, which meant it applied to the remote case, and I suspect that results in a stat() call against a URL that fails and causes it to bypass the reload code every time.
Scott Cantor November 16, 2017 at 9:30 PM
This "fix" breaks remote resource reloads, so I've reverted it for a 1.6.2 release and we'll need to pull it from the master branch.
Rod Widdowson February 16, 2017 at 10:49 AM
6a16f05 changes the comments
I briefly toyed with annotating all the locking, but there isn't s trace level and it would be incredibly noisy at debug.
Calling this one done.
Fixed
Pinned fields
Click on the next to a field label to start pinning.
Remote reload is unaffected with the latest fix.