fixed saving bug

This commit is contained in:
Tom Needham 2012-01-09 17:03:19 +00:00
parent f7f19af816
commit 18b8dc1c21
2 changed files with 13 additions and 3 deletions

View File

@ -40,7 +40,8 @@ if($path != '' && $mtime != '')
if($mtime != $filemtime) if($mtime != $filemtime)
{ {
// Then the file has changed since opening // Then the file has changed since opening
OC_JSON::error(); OC_JSON::error();
OC_Log::write('files_texteditor',"File: ".$path." modified since opening.",OC_Log::ERROR);
} }
else else
{ {
@ -49,14 +50,20 @@ if($path != '' && $mtime != '')
if(OC_Filesystem::is_writeable($path)) if(OC_Filesystem::is_writeable($path))
{ {
OC_Filesystem::file_put_contents($path, $filecontents); OC_Filesystem::file_put_contents($path, $filecontents);
OC_JSON::success(); // Clear statcache
clearstatcache();
// Get new mtime
$newmtime = OC_Filesystem::filemtime($path);
OC_JSON::success(array('data' => array('mtime' => $newmtime)));
} }
else else
{ {
// Not writeable! // Not writeable!
OC_JSON::error(array('data' => array( 'message' => 'Insufficient permissions'))); OC_JSON::error(array('data' => array( 'message' => 'Insufficient permissions')));
OC_Log::write('files_texteditor',"User does not have permission to write to file: ".$path,OC_Log::ERROR);
} }
} }
} else { } else {
OC_JSON::error(array('data' => array( 'message' => 'File path or mtime not supplied'))); OC_JSON::error(array('data' => array( 'message' => 'File path or mtime not supplied')));
OC_Log::write('files_texteditor',"Invalid path supplied:".$path,OC_Log::ERROR);
} }

View File

@ -102,6 +102,9 @@ function doFileSave(){
}, 2000); }, 2000);
} else { } else {
// Save OK // Save OK
// Update mtime
$('#editor').attr('data-mtime',jsondata.data.mtime);
// Show result
$('#saving_icon').remove(); $('#saving_icon').remove();
$('#editor_save').after('<p id="save_result" style="float: left">Saved</p>') $('#editor_save').after('<p id="save_result" style="float: left">Saved</p>')
setTimeout(function() { setTimeout(function() {