OBM
  1. OBM
  2. OBMFULL-6448

Duplicating a moved exception screws up the repeatkind

    Details

    • Story Points:
      2
    • Similar Issues:
      Show 10 results 
    • Rank:
      6924
    • Sprint:
      2016#01

      Description

      Reproduction steps

      1. As usera with obm-ui, create a weekly recurrent event (no matter the attendees and other options).
      2. As usera with obm-ui, edit one occurrence so that it becomes a moved exception.
      3. As usera with obm-ui, duplicate (yeah, this feature exists) the moved exception, changing the time. Make sure you check 'Force insertion'

      Current behavior

      1. The event is not visible in obm-ui. NOK
      2. TB synchronization is broken at this point. NOK

      Expected behavior

      1. The event is visible in obm-ui. OK
      2. TB synchronization works as expected. OK

      Additional informations

      1. The event has no repeatkind in database
      obm=# select event_id, event_title, event_date, event_repeatkind from event order by event_id desc limit 3;
       event_id | event_title  |     event_date      | event_repeatkind 
      ----------+--------------+---------------------+------------------
         126558 | Sdfds dfsfss | 2016-01-13 13:00:00 | 
         126557 | Sdfds        | 2016-01-13 09:00:00 | none
         126556 | Sdfds        | 2016-01-13 07:00:00 | none
      (3 rows)
      

        Issue Links

          Activity

          Hide
          David Dolcimascolo added a comment -

          To QA:

          • Impacts obm-ui, when duplicating an event.
            • Duplicating a moved exception now works as expected. You'll notice the block "Repeat" has changed, you can now see the correct block where you can select the repeat kind, the repeat days, etc.
          • To test, follow the steps to reproduce
            • You can test duplicating a series
            • You can test duplicating a simple event
            • Make sure TB sync does not break with every test

          Tested on CentOS7, with TB38.

          Show
          David Dolcimascolo added a comment - To QA: Impacts obm-ui , when duplicating an event. Duplicating a moved exception now works as expected. You'll notice the block "Repeat" has changed, you can now see the correct block where you can select the repeat kind, the repeat days, etc. To test, follow the steps to reproduce You can test duplicating a series You can test duplicating a simple event Make sure TB sync does not break with every test Tested on CentOS7, with TB38.
          Hide
          Jenkins Continuous Integration Server added a comment -

          SUCCESS: Integrated in poll-master #220
          OBMFULL-6448 Do not insert an empty repeat_kind when duplicating exceptions. (ddolcimascolo: 714789a11ba7ffec40cf958faea426ca0b423fe1)

          • ui/php/calendar/calendar_display.inc
          • ui/php/calendar/calendar_query.inc
          Show
          Jenkins Continuous Integration Server added a comment - SUCCESS: Integrated in poll-master #220 OBMFULL-6448 Do not insert an empty repeat_kind when duplicating exceptions. (ddolcimascolo: 714789a11ba7ffec40cf958faea426ca0b423fe1 ) ui/php/calendar/calendar_display.inc ui/php/calendar/calendar_query.inc
          Hide
          Stephane COLSON added a comment -

          OK, tested with OBM 3.2.0~0.alpha0+git20160419-1747-3fc3c43 & Opush 3.1.2~rc0-1 with ui, TB31 & TB38, iPhone 5 (iOS 9), Fairphone 2 (Android 5) and SGS6 (Android 6)
          With daily, weekly, allday or not recurrent event with or without end date. Duplicate all the serie, or only an exception...

          Show
          Stephane COLSON added a comment - OK, tested with OBM 3.2.0~0.alpha0+git20160419-1747-3fc3c43 & Opush 3.1.2~rc0-1 with ui, TB31 & TB38, iPhone 5 (iOS 9), Fairphone 2 (Android 5) and SGS6 (Android 6) With daily, weekly, allday or not recurrent event with or without end date. Duplicate all the serie, or only an exception...

            People

            • Assignee:
              David Dolcimascolo
              Reporter:
              David Dolcimascolo
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Agile