1. 18 Jan, 2019 3 commits
  2. 09 Jan, 2019 1 commit
  3. 08 Jan, 2019 1 commit
    • Bea Lam's avatar
      [commhistoryd] Refine group matching when there are multiple recipients. Contributes to JB#42099 · 08001bb0
      Bea Lam authored
      When slotGroupInserted() is called, prefer to assign the current group
      based on an exact recipient match rather than matching the first group
      that contains any matching recipients.
      
      This was already implemented for slotOnModelReady() by commit
      1084b380 but needs to be done for
      slotGroupInserted() as well.
      
      This fixes an issue where if there are multiple message recipients, it
      matches the group id for the broadcast message rather than the
      individual message. Thus, the wrong group id is used when searching
      the database for corresponding events, and so events cannot be located
      and updated as expected.
      08001bb0
  4. 07 Jan, 2019 1 commit
  5. 21 Dec, 2018 1 commit
    • Bea Lam's avatar
      [commhistoryd] Don't use getEventById() result if no events are found. Contributes to JB#42099 · e91fb669
      Bea Lam authored
      getEventById() returns true if the database call succeeded, and
      not necessarily whether a valid event was found.
      
      The main issue here is that TextChannelListener::slotMessageSent()
      was using an invalid event returned by getEventById(), filling that
      with valid data and then failing on the saveMessage() call because
      the event is actually invalid, and so the event with the updated
      messageToken, status etc. is never saved to the db.
      
      Most other calls to getEventById() already check whether it found
      a valid event. The patch updates some other calls in MmsHandler
      that were not checking this, to make sure those don't cause any
      unintended side effects.
      e91fb669
  6. 02 Jul, 2018 2 commits
  7. 10 Apr, 2018 2 commits
  8. 30 Jan, 2017 2 commits
    • Slava Monich's avatar
      [rpm] Require libcommhistory-qt5 >= 1.9.33 · 15638dee
      Slava Monich authored
      15638dee
    • Slava Monich's avatar
      Merge branch 'dup' into 'master' · 11f7fc14
      Slava Monich authored
      Ignore duplicate MMS notifications
      
      If the user doesn't download the MMS message right away, operators
      often resend the push notification. These were treated as separate
      messages and littered the conversation view.
      
      Now we use mmsId field to store the location url from the MMS push
      notification. It's cleared as soon as the message gets downloaded.
      If we receive a push message while another MMS with the same URL
      is waiting for download, we ignore the duplicate.
      
      Requires a few changes in mms-engine as well. Both old and new variants
      of org.nemomobile.MmsHandler.messageNotification are supported (for now).
      The new one includes the location URL, so that commhistoryd doesn't have
      to parse the MMS PDU.
      
      See merge request !23
      11f7fc14
  9. 26 Jan, 2017 1 commit
    • Slava Monich's avatar
      [commhistory-daemon] Ignore duplicate MMS notifications. Contributes to JB#18573 · c8bcab6e
      Slava Monich authored
      If the user doesn't download the MMS message right away, operators
      often resend the push notification. These were treated as separate
      messages and littered the conversation view.
      
      Now we use mmsId field to store the location url from the MMS push
      notification. It's cleared as soon as the message gets downloaded.
      If we receive a push message while another MMS with the same URL
      is waiting for download, we ignore the duplicate.
      
      Requires a few changes in mms-engine as well. Both old and new variants
      of org.nemomobile.MmsHandler.messageNotification are supported (for now).
      The new one includes the location URL, so that commhistoryd doesn't have
      to parse the MMS PDU.
      c8bcab6e
  10. 19 Aug, 2016 2 commits
  11. 18 Aug, 2016 2 commits
    • sletta's avatar
      Make custom moc compilation pass. · 22c9b20c
      sletta authored
      We're running a manual moc compilation pass on generated source files
      and we're picking up INCPATH from the generated Makefile as there
      seems to be no other way to get access to the complete include path.
      
      However, since 0b144bc76a368ecc6c5c1121a1b51e888a0621ac, the includepath
      contains -isystem rather than -I for many of the include paths, which
      moc fails to parse. So call moc in a wrapper shell that fixes -isystem
      to -I arguments.
      22c9b20c
    • sletta's avatar
      Conform to stricter warning levels · 6e6ea14f
      sletta authored
      6e6ea14f
  12. 19 Apr, 2016 2 commits
  13. 18 Apr, 2016 1 commit
  14. 08 Mar, 2016 3 commits
  15. 04 Mar, 2016 1 commit
    • pvuorela's avatar
      Merge branch 'master' into 'master' · 6bd177f2
      pvuorela authored
      [commhistory-daemon] Avoid locale specific numbers in debug output. Contributes to MER#1536
      
      Minor detail as debug needs to be explicitly turned on. Can be tested by running
      LC_ALL=mr_IN.utf8 commhistoryd -d
      -> journal gets a few lines 
      
      @mvogt 
      
      See merge request !17
      6bd177f2
  16. 03 Mar, 2016 3 commits
  17. 26 Feb, 2016 2 commits
  18. 25 Feb, 2016 10 commits