Menu

#3115 Bugs with Europe screen

Fixed_0.12.0
closed-fixed
nobody
None
5
2022-05-01
2017-11-02
Kurzov
No

It will not let me drag the maritime unit out of the "at harbor" box into the "sailing for new world" box.

Save attached.

1 Attachments

Discussion

  • Adrian Schroeter

    I think I found the issue.

    In the code we throw a NPE because the owner of the unit does not have a entryLocation set.

     
    • Mike Pope

      Mike Pope - 2017-11-06

      The entryLocation issue is known and partly understood. I hope to beback to working on it by the weekend.

       
    • Mike Pope

      Mike Pope - 2017-11-11

      git.8ac53e7 should fix it. Did that work for you?

       
  • Mike Pope

    Mike Pope - 2017-11-11
    • status: open --> open-needs-info
     
  • Adrian Schroeter

    I don't see git.8ac53e7

    do you mean
    https://github.com/FreeCol/freecol/pull/7

    that works for me

     
  • Kurzov

    Kurzov - 2017-11-14

    Gah, I am sorry, I completely forgot about this. I've got a lot of things to do.

    Regardless, I'll test this out sometime this week.

     
  • wintertime

    wintertime - 2019-12-07

    I just had this problem with a new game on git.f4aca10f, to not be able to sail back to the new world. I only got server can not do that and there is the warning of the high seas not being connected inside the log.

     
  • wintertime

    wintertime - 2019-12-20
     
  • Marcin

    Marcin - 2019-12-20

    I can test it no problem. However I normally start from Weekly Releases which are ready to roll. I am not sure how to test one fix. It looks like it was merged after last weekly so once next weekly is created I can test from there.

     

    Last edit: Marcin 2019-12-20
  • wintertime

    wintertime - 2019-12-22

    I merged GitHub PR#45 now, as it helped. If your ship is already in Europe it may still be bugged.
    You can use the new nightly I just started for further testing.

     
  • Marcin

    Marcin - 2019-12-23

    Tested in new Weekly. It works fine. Will do few more test and confirm but initial one went ok.

     
  • Marcin

    Marcin - 2020-01-02

    Tested in 2019-12-29. Fixed.

     
  • Mike Pope

    Mike Pope - 2020-01-03

    Many thanks for the testing Marcin. I was up to about BR#3142 when I had to step back from FreeCol work last year, so checking older bugs than that (like this one) is the most helpful, as they are the ones that are either believed fixed or needing more information. I am just starting to work on BR#3142 today.

     
  • wintertime

    wintertime - 2020-01-14

    This is related: https://sourceforge.net/p/freecol/bugs/3167/
    Someone commented there he did some bisecting on the bug and found the commit.

     
  • Mike Pope

    Mike Pope - 2020-01-25

    The next fix for this (and BR#3167) is in git.58660f4a. The patch survived overnight testing but there are some things I am not sure about still.

     
  • Mike Pope

    Mike Pope - 2022-04-24
    • status: open-needs-info --> pending-fixed
    • Group: Current --> Fixed_trunk
     
  • Mike Pope

    Mike Pope - 2022-04-24

    Looks like it worked, setting to Pending.

     
  • Mike Pope

    Mike Pope - 2022-05-01
    • Group: Fixed_trunk --> Fixed_0.12.0
     
  • Mike Pope

    Mike Pope - 2022-05-01
    • Status: pending-fixed --> closed-fixed
     

Log in to post a comment.