Cache and re-use a lineitem?

[cmcfadden]

Curious to sanity check something before doing a fork.

We have an app that will create multiple lineitems, and then may be updating them frequently. Rather than going back to the LMS continually to fetch that info, I'd like to just cache those in our app.

Right now, the AGS put_grade method will always reload a lineitem, unless it was the claim/endpoint.

I was thinking of adding a new conditional in put_grade line 22

} else if($lineitem !== null && !empty($lineitem->get_id()) {
$score_url = $lineitem->get_id();
}

Anyone see a problem with doing that?

[kpommerenke]

We've been using the exact same two lines in production for two months and have not had any issues.

What did create an issue was to get the launch data from cache based on the launch ID stored in the session, when students were able to switch from one course to another in our tool without relaunching from the LMS, as the launch data from the session then showed the wrong context ID and picked the wrong lineitem URL.