The player's current character id is set in two ways: via the URL parameter (pc ) or via the session.pc field. The session is the fallback if no pc.id is set on the URL.
Task: change the BaseController.fetchPc method to only use the session's field as the method of loading the PlayerCharacter from the database, unless necessary (like in TownController.show()).
UserAccount should get a field like "lastCharacterId" in case a user logs in on a sub page (like shop/show). This way it should be possible to load the correct character for this page. (Unless someone uses multiple chars and logs in with an old browser tab, but since pages cannot be guaranteed to be fully reentrant {example: fight pages}, this seems like a small price to pay.)
The player's current character id is set in two ways: via the URL parameter (pc ) or via the session.pc field. The session is the fallback if no pc.id is set on the URL.
Task: change the BaseController.fetchPc method to only use the session's field as the method of loading the PlayerCharacter from the database, unless necessary (like in TownController.show()).
UserAccount should get a field like "lastCharacterId" in case a user logs in on a sub page (like shop/show). This way it should be possible to load the correct character for this page. (Unless someone uses multiple chars and logs in with an old browser tab, but since pages cannot be guaranteed to be fully reentrant {example: fight pages}, this seems like a small price to pay.)