repeating events don't work

[mschulze0]

I have a very similar setup to the non-working setup as in #18 (except that I have some repeating instruments in the first event) and @DEFAULT-FROM-PREVIOUS-EVENT is not working for me when I use it in a repeating event.

as a test I just copied the project and disabled all repeating instruments in my first event so that I now have the same structure as #18. Still doesn't work.

[mschulze0]

Hi,

looking through the code I noticed that the module uses the previous event even when I want to use values from previous instances from the same event.

                    foreach ($events as $event) {
                        //if ($event == $_GET['event_id']) {
                        //    break;
                        //}
                        if (in_array($source_form, $Proj->eventsForms[$event])) {
                            $prev_event = $event;
                        }
                        if ($event == $_GET['event_id']) {
                            break;
                        }
                    }

by putting the if at the end of the loop it now works with repeating events, but obviously this breaks getting values from the previous event for non-repeating events.

[ChemiKyle]

Hi @mschulze0

Could you please share your Project XML? You'll have to zip it to share it here in github.

[mschulze0]

@ChemiKyle unfortunately I cannot. You can't replicate it?

[mschulze0]

@ChemiKyle any news?

[ChemiKyle]

Hi @mschulze0, sorry I've had my hands full with COVID related work.
@mbentz-uf do you think you can take a look at this? The information scattered around issue #18 and PR #24 (namely the project XML) may be helpful. FRSL can be annoying to configure; I have an experimental module that exports EM configs, so please do let me know if that's a blocker.

[michael-bentz]

I was able to replicate this. The module works for Repeat Entire Event (repeat all instruments together) but is breaking for Repeat instruments (repeat independently of each other) (Enrollment Event below)

I'm looking into adding support for event instances.

[mschulze0]

@mbentz-uf would love to test once you implement a fix :)