
<html><head></head><body>No, this is all on the Mac. <br><br><div class="gmail_quote">On June 28, 2019 1:56:08 PM GMT+09:00, Berthold Stoeger <bstoeger@mail.tuwien.ac.at> wrote:<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">Hi Dirk,<br><br>On Friday, 28 June 2019 06:00:44 CEST Dirk Hohndel wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Every time I download a new dive from my dive computer, Subsurface creates a<br>new trip for the downloaded dive instead of adding the dive to the existing<br>dive trip.<br></blockquote><br>Miika reported the same problem.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Reading the code I can see that we have two very different paths for<br>"addDive":<br><br>DiveListBase::addDive() which is called when importing from a dive computer<br>- which doesn't even try to match the dive to an existing trip<br>AddDive::addDive() which appears to do the right thing, but creates an undo<br>event per dive that's added (which isn't what we want for the import).<br></blockquote><br>Dive import on desktop works via Command::importDives(). Whether a new trip is <br>generated is controlled with the IMPORT_ADD_TO_NEW_TRIP flag. See desktop-<br>widgets/downloadfromdivecomputer.cpp.<br><br>I reckon that you download with your phone? I can try to switch that to <br>Command::importDives() but I think I would have to do this "blind", i.e. <br>without the possibility to test. If I remember correctly, BT does not work on <br>my mobile-on-desktop version.<br><br>Berthold<br><br><br></pre></blockquote></div><br>-- <br>From my phone</body></html>