The schedules assigned to the products are using the ids 1,2,3,4 in the alphabetical order, not the REAL id's of those schedules internally,
so if you have these schedules

1 daily renewal
2 4weekly-renewal

the form will display them in order

4weekly-renewal
daily renewal

and whenever you create a product with "daily renewal" you get sid=2 into the config (AKA 4weekly)
just because it'S the 2nd line in that form

rendering this function defect for all cases with > 1 schedule

Comments

spazfox’s picture

spazfox
Location Olympia, WA
commented

I can confirm this behavior; I'm having the exact same problem on my site. Seems like a relatively easy thing to fix -- anyone working on this?

jonathan_hunt’s picture

jonathan_hunt commented

I encountered the same issue. In ec_recurring_form_alter() I changed

$slist = array_merge(array(0 => '--'), ec_recurring_get_list($allow_renewals));

to

$slist = array(0 => '--') + ec_recurring_get_list($allow_renewals);

It's working better so far. See use of + operator according to http://au2.php.net/array_merge

brmassa’s picture

brmassa commented
Status: Active » Fixed

Guys,

its fixed on eC4

regards,

massa

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.