From: Erik V. <eri...@hc...> - 2005-12-26 11:20:46
|
> Wow. Now I can see why you got lost. You've nested the > token laying calls quite deeply. I think much of this could > be simplified, but for now it works with the one-line change > I've committed. > > In PublicCompany.layBaseToken() you were adding the token to > the company, but weren't adding the token to the hex. After > including a call to hex.addToken(), the token is being drawn > on the map. But still only the very first laid non-home token of any company is shown! (I have not mentioned this before, but this was also the case with your original token laying code). BTW the station number in the call you added was 1 but should have been 0: I got an exception. And the token is still only drawn after "Done" has been pressed. My idea was to draw the token as soon as the hex was clicked in token laying mode. This would show a "provisional token", just as with tile laying, to be replaced with a permanent token when Done is pressed. > Now we just need to add in the logic for selecting the > station for hexes with multiple stations. I think I will start with: - implementing the keep-existing-track rule for tile laying; this will require me to get deeper into track and station handling; - fixing the tile orientations as displayed in the OR window. From there I think linking in the station selection for token laying will be easier. Erik. |