openwebwork / webwork2

Course management front end for WeBWorK
http://webwork.maa.org/wiki/Main_Page
Other
145 stars 165 forks source link

Fix a warning that occurs for proctor authentication. #2347

Closed drgrice1 closed 7 months ago

drgrice1 commented 7 months ago

This is a stopgap measure since this is already fixed in #2334, but this was missed in #2333. #2334 completely rewrites this module.

The WeBWorK::Authen::create_session method no longer accepts a prior database key object for its second argument, so the optional third argument (which is now the optional second argument) is in the wrong place for the WeBWorK::Authen::Proctor modules override of this method.

Alex-Jordan commented 7 months ago

I'll merge this so it won't somehow get in the way of someone testing before and after with #2334.