- Posts: 26
Shop / e-Commerce Integration
Remove check from joomdle3/joomdle3/app/joomdle/joomdle.php
- Ahmad Arshad
- Topic Author
- Offline
- Banned
-
Less
More
8 years 1 week ago #1
by Ahmad Arshad
Remove check from joomdle3/joomdle3/app/joomdle/joomdle.php was created by Ahmad Arshad
Antonio.
Does it make sense to remove this check from /plugins/payplans/joomdle3/joomdle3/app/joomdle/joomdle.php
public function onPayplansSubscriptionAfterSave($prev, $new)
{
// no need to trigger if previous and current state is same
if($prev != null && $prev->getStatus() == $new->getStatus()){
return true;
}
I have a large volume of subscriptions in payplans... which sometimes for some reason do not create enrollments on moodle.... I have to go in each of them and just save them again within payplans... If this check is there, it doesnt trigger a change...
Or maybe a more batch way of syncing enrollments just as user info gets updated
Does it make sense to remove this check from /plugins/payplans/joomdle3/joomdle3/app/joomdle/joomdle.php
public function onPayplansSubscriptionAfterSave($prev, $new)
{
// no need to trigger if previous and current state is same
if($prev != null && $prev->getStatus() == $new->getStatus()){
return true;
}
I have a large volume of subscriptions in payplans... which sometimes for some reason do not create enrollments on moodle.... I have to go in each of them and just save them again within payplans... If this check is there, it doesnt trigger a change...
Or maybe a more batch way of syncing enrollments just as user info gets updated
Please Log in or Create an account to join the conversation.
- Antonio Durán
-
- Offline
- Moderator
-
Less
More
- Posts: 7914
8 years 1 week ago #2
by Antonio Durán
Replied by Antonio Durán on topic Remove check from joomdle3/joomdle3/app/joomdle/joomdle.php
Hi.
Yes, I think you can remove it without causing any issues.
> Or maybe a more batch way of syncing enrollments just as user info gets updated
There is no way to do that right now. Instead of trying to create it, I think it would be better to try to find out why some enrolments are failing.
Yes, I think you can remove it without causing any issues.
> Or maybe a more batch way of syncing enrollments just as user info gets updated
There is no way to do that right now. Instead of trying to create it, I think it would be better to try to find out why some enrolments are failing.
Please Log in or Create an account to join the conversation.