[ 
https://issues.apache.org/jira/browse/VCL-1001?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15634034#comment-15634034
 ] 

ASF subversion and git services commented on VCL-1001:
------------------------------------------------------

Commit 1767960 from [~jfthomps] in branch 'vcl/trunk'
[ https://svn.apache.org/r1767960 ]

VCL-1001 - changing a future reservation to a now reservation doesn't properly 
update the reserved computer

requests.php: modified AJsubmitEditRequest: added code to determine if old 
start time was in the future or not and pass 'now' or 'future' to updateRequest 
accordingly

utils.php: modified updateRequest: added additional argument $nowfuture and 
removed code that determined if the reservation was now or future based on the 
reservation start time - the problem was that if a future reservation got 
changed to a now reservation, the isAvailable could have changed the 
computerid, but nowfuture was determined to be now and then the 2nd half of the 
function where the computerid is updated was skipped; so the user ended up with 
a reservation on the same computer they had, which may or may not have another 
reservation on it

xmlrpcWrappers.php: updated XMLRPCextendRequest and XMLRPCsetRequestEnding: 
added 'now' to call to updateRequest

> changing a future reservation to a now reservation doesn't properly update 
> the reserved computer
> ------------------------------------------------------------------------------------------------
>
>                 Key: VCL-1001
>                 URL: https://issues.apache.org/jira/browse/VCL-1001
>             Project: VCL
>          Issue Type: Bug
>          Components: web gui (frontend)
>            Reporter: Josh Thompson
>             Fix For: 2.5
>
>
> If a user has a reservation for a computer that is in the future and then 
> changes the reservation so that the start time is in the past, making it a 
> "now" reservation, the code that updates which computer is assigned to the 
> user gets skipped. Fortunately, the backend catches that the computer is 
> inuse and fails the reservation.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to