Maptool: Vision for Non Individual View Broken

[Phergus]

MT Version 1.5.0 - Both installed build and JAR only build with Java 10.
Also seen in Nerps 1.4.4.0, 1.4.5.4 and MT 1.3.b87, 1.3.b89, 1.3.b91, 1.4.0.5.
OS: Win10x64

With a server running and server option Individual View _not enabled_ there are two problems.

First problem is that with Player View enabled on the GM client, no PC or NPC tokens are visible. Selecting a token prior to switching to Player View has no effect on this.

Second problem is that the Player Clients are getting Individual Views and not a shared PC view. With an owned token selected, player clients should get that token's individual view. With no tokens selected the player clients should display the combined view visible to all PC tokens . Related is that a player client with no token ownership does get the all PCs view.

The expected view for Player Clients is what is seen in MapTool with no server running and no connection to a server. With Player View enabled and no tokens selected, all tokens within the viewable area of any PC tokens are shown. With a single token selected, only those tokens within the selected token's viewable area are shown.

Top-Left: GM Client with Player View enabled
Top-Right: Player Client with no token ownership
Bottom-Left: Player Client with ownership of Elf token
Bottom-Right: Player Client with ownership of Mystic token

Campaign file used in screenshots.
VisionProblem1.cmpgn.zip

[dorpond]

I just tested in Nerps 1.4.5.3 and the issue still exists there. I know I worked closely with Jamz on vision and lighting back in Nerps 1.4.4.0, and he eliminated a bunch of bugs, but to be honest, I tested individual views more than not, but nevertheless, I'd like to see if this issue was still in 1.4.4.0. Jamz did some change lighting/Vision in 1.4.5.0, so it will be nice to know if he introduced that bug or if it was there in 1.4.4.0.

Previous Jamz versions can be downloaded here in case anyone wishes to test 1.4.4.0 before I get a chance. It could provide a clue:
https://github.com/JamzTheMan/MapTool/releases

[Phergus]

It's broken exactly the same way in 1.4.4.0. So most likely this was broken in 1.4.0.5.

[Phergus]

Also broken in 1.3.b87, 1.3.b89 and 1.3.b91. May have been broken forever.

[Azhrei]

@Phergus:

Second problem is that the Player Clients are getting Individual Views and not a shared PC view. With an owned token selected, player clients should get that token's individual view.

I don't understand. Did you mean, "player clients should NOT get that token's individual view"? Assuming you did...

First problem is that with Player View enabled on the GM client, no PC or NPC tokens are visible.

Yep, that's clearly a problem. Surprised this hasn't come up before, given that you found it's been like this since 1.3b87!

Related is that a player client with no token ownership does get the all PCs view.

Which makes sense to me, since there's no individual views.

The expected view for Player Clients is what is seen in MapTool with no server running and no connection to a server. With Player View enabled and no tokens selected, all tokens within the viewable area of any PC tokens are shown. With a single token selected, only those tokens within the selected token's viewable area are shown.

IIRC, when there is no server running, there is no concept of Player vs. GM, so the Player View option has nothing to work with.

It sounds like there might be a missing isPlayer() and/or isGM() in the code that determines which tokens to display. Digging through that code is likely to take some time. There are a lot of interactions between that code and other parts of MapTool (like the pointer tool rendering templates or the network layer tracking the movement of remote tokens)...

In any case, good catch on detecting the problem!

[JamzTheMan]

Woot, I didn't break it this time lol

So honestly if it's been broken since b87 I probably won't touch it in favor of moving on to 2.0...

[Phergus]

@Azhrei

Second problem is that the Player Clients are getting Individual Views and not a shared PC view. With an owned token selected, player clients should get that token's individual view.

I don't understand. Did you mean, "player clients should NOT get that token's individual view"? Assuming you did...

The server has Individual View turned off. The player client, _with no tokens selected_, should get the combined view of all Player tokens, owned or not. This would match the view in the Top-Right. Instead, with the existing code, the player logged in as Elf only sees what the Elf token sees. If MT was working correctly, and the Elf player selects an owned token, then they should get only the view of that token and not the combined view. Currently, that is the view the player gets whether the token is selected or not. See bottom-left and bottom-right. This is wrong.

Related is that a player client with no token ownership does get the all PCs view.

Which makes sense to me, since there's no individual views.

That is a contrasting statement showing that a client without any token ownership gets the correct view (Top-Right) while the client that actually owns a token on the map does not.

It would be best if you just fired up 2 or 3 MT clients and looked at it.

[Merudo]

The first issue (no vision when Player View enabled on the GM's client) is indeed a problem.

However, I'm having no trouble at all with the second issue. The players can see everything just fine.

EDIT: I figured it out, the bug doesn't happen when Strict Token Ownership is off.

[Merudo]

My commit fixes the first issue.

[Merudo]

My second commit fixes the second issue. Non Individual View should work fine now.

[Phergus]

Testing Server with Individual View unchecked:

• [x] GM Client - View as Player
• [x] GM Client - View as GM
• [x] Player Client - With no token ownership
• [x] Player Client - With single token ownership (Elf) - No token selected
• [x] Player Client - With single token ownership (Elf) - Owned token selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - No selected tokens
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Mystic selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Eagle selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Both selected

Testing Server with Individual View checked:

• [x] GM Client - View as Player
• [x] GM Client - View as GM
• [x] Player Client - With no token ownership
• [x] Player Client - With single token ownership (Elf) - No token selected
• [x] Player Client - With single token ownership (Elf) - Owned token selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - No selected tokens
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Mystic selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Eagle selected
• [x] Player Client - With multiple token ownership (Mystic, Eagle) - Both selected

Testing No Server:

• [x] GM Client - View as Player
• [x] GM Client - View as GM

[Phergus]

All seems to be working as expected.