type cast $selected variable on restore

[pjosephson]

You will need to cast $selected variable (around line 219) on the restore or you may get an empty character which makes the file name incorrect.

$file = new File($this->folder . $config . DS . $selected . '' . $versions[(int)$selected]['file']);
to
$file = new File($this->folder . $config . DS . (int) $selected . '' . $versions[(int)$selected]['file']);

[thatcode]

I believe this is incorrect.

if ($this->quiet) {
$selected = (string)max(array_keys($versions));
} else {
$options = array_keys($versions);
$options[] = 'Q';
$selected = strtoupper($this->in('Which version would you like to restore?', $options));
}

If ($this->quiet) then $selected is one of the keys of the $versions array. We already know that this is not empty.
Else, we ask the user for an input. Cake does not allow an answer not in the list (if one is given) subject to case-insensitivity. Therefore we know it is either in the $versions array, or Q. If it's not Q, it's in the versions array.

I'm going to leave this open for now, but I suspect I'll close it soon without a POC or explanation of how you came across that code.

Thatcode