Add support to windows unc remote path in module File; code fix/enhancement in KVS classes

1 files changed, 120 insertions, 99 deletions

diff --git a/src/modules/file/libkvifile.cpp b/src/modules/file/libkvifile.cpp
index 2ff9022f1..8220e74bf 100644
--- a/src/modules/file/libkvifile.cpp
+++ b/src/modules/file/libkvifile.cpp
@@ -94,8 +94,11 @@ static bool file_kvs_cmd_copy(KviKvsModuleCommandCall * c)
 		KVSM_PARAMETER("source",KVS_PT_NONEMPTYSTRING,0,szSrc)
 		KVSM_PARAMETER("destination",KVS_PT_NONEMPTYSTRING,0,szDst)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szSrc);
-	KviFileUtils::adjustFilePath(szDst);
+
+	if (szSrc.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szSrc);
+	if (szDst.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDst);
 	if(KviFileUtils::fileExists(szDst) && !c->switches()->find('o',"overwrite"))
 	{
 		c->warning(__tr2qs("Destination file exists: no copy made"));
@@ -133,7 +136,8 @@ static bool file_kvs_cmd_addimagepath(KviKvsModuleCommandCall * c)
 	KVSM_PARAMETERS_BEGIN(c)
 		KVSM_PARAMETER("path",KVS_PT_NONEMPTYSTRING,0,szDst)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szDst);
+	if (szDst.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDst);
 
 	//QStringList::Iterator it = KVI_OPTION_STRINGLIST(KviOption_stringlistImageSearchPaths).find(szDst);
 	int iIndex = KVI_OPTION_STRINGLIST(KviOption_stringlistImageSearchPaths).indexOf(szDst);
@@ -166,7 +170,8 @@ static bool file_kvs_cmd_delimagepath(KviKvsModuleCommandCall * c)
 	KVSM_PARAMETERS_BEGIN(c)
 		KVSM_PARAMETER("path",KVS_PT_NONEMPTYSTRING,0,szDst)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szDst);
+	if (szDst.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDst);
 
 	//QStringList::Iterator it = KVI_OPTION_STRINGLIST(KviOption_stringlistImageSearchPaths).find(szDst);
 	int iIndex = KVI_OPTION_STRINGLIST(KviOption_stringlistImageSearchPaths).indexOf(szDst);
@@ -215,8 +220,8 @@ static bool file_kvs_cmd_write(KviKvsModuleCommandCall * c)
 		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 		KVSM_PARAMETER("data",KVS_PT_NONEMPTYSTRING,0,szData)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szFileName);
-
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 	bool bRet;
 	if(c->switches()->find('l',"local-8-bit"))
 		bRet = KviFileUtils::writeFileLocal8Bit(szFileName,szData,c->switches()->find('a',"append"));
@@ -272,7 +277,8 @@ static bool file_kvs_cmd_writeBytes(KviKvsModuleCommandCall * c)
 		KVSM_PARAMETER("dataArray",KVS_PT_ARRAY,0,pArray)
 	KVSM_PARAMETERS_END(c)
 
-	KviFileUtils::adjustFilePath(szFileName);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 
 	if(!pArray)
 		return c->error(__tr2qs("Missing data array"));
@@ -339,8 +345,10 @@ static bool file_kvs_cmd_rename(KviKvsModuleCommandCall * c)
 		KVSM_PARAMETER("newname",KVS_PT_NONEMPTYSTRING,0,szNew)
 	KVSM_PARAMETERS_END(c)
 
-	KviFileUtils::adjustFilePath(szOld);
-	KviFileUtils::adjustFilePath(szNew);
+	if (szOld.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szOld);
+	if (szNew.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szNew);
 
 	if(KviFileUtils::fileExists(szNew))
 		c->warning(__tr2qs("Destination file exists: file not renamed"));
@@ -382,7 +390,8 @@ static bool file_kvs_cmd_mkdir(KviKvsModuleCommandCall * c)
 	KVSM_PARAMETERS_BEGIN(c)
 		KVSM_PARAMETER("directory",KVS_PT_NONEMPTYSTRING,0,szDir)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szDir);
+	if (szDir.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDir);
 	if(!KviFileUtils::makeDir(szDir))
 	{
 		if(!c->switches()->find('q',"quiet"))
@@ -418,15 +427,16 @@ static bool file_kvs_cmd_mkdir(KviKvsModuleCommandCall * c)
 
 static bool file_kvs_cmd_remove(KviKvsModuleCommandCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("name",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("name",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	if(!KviFileUtils::removeFile(szName))
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
+	if(!KviFileUtils::removeFile(szFileName))
 	{
 		if(!c->switches()->find('q',"quiet"))
-			c->warning(__tr2qs("Failed to remove the file '%Q'"),&szName);
+			c->warning(__tr2qs("Failed to remove the file '%Q'"),&szFileName);
 	}
 	return true;
 }
@@ -460,15 +470,16 @@ static bool file_kvs_cmd_remove(KviKvsModuleCommandCall * c)
 
 static bool file_kvs_cmd_rmdir(KviKvsModuleCommandCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("name",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("name",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	if(!KviFileUtils::removeDir(szName))
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
+	if(!KviFileUtils::removeDir(szFileName))
 	{
 		if(!c->switches()->find('q',"quiet"))
-			c->warning(__tr2qs("Failed to remove the directory '%Q'"),&szName);
+			c->warning(__tr2qs("Failed to remove the directory '%Q'"),&szFileName);
 	}
 	return true;
 }
@@ -497,12 +508,13 @@ static bool file_kvs_cmd_rmdir(KviKvsModuleCommandCall * c)
 
 static bool file_kvs_fnc_exists(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	QFileInfo f(szName);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
+	QFileInfo f(szFileName);
 	c->returnValue()->setBoolean(f.exists());
 	return true;
 }
@@ -525,12 +537,13 @@ static bool file_kvs_fnc_exists(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_type(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	QFileInfo f(szName);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
+	QFileInfo f(szFileName);
 	if(f.isFile())
 		c->returnValue()->setString("f");
 	else if(f.isDir())
@@ -558,12 +571,13 @@ static bool file_kvs_fnc_type(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_size(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	QFileInfo f(szName);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
+	QFileInfo f(szFileName);
 	c->returnValue()->setInteger(f.size());
 	return true;
 }
@@ -589,7 +603,8 @@ static bool file_kvs_fnc_allSizes(KviKvsModuleFunctionCall * c)
 	KVSM_PARAMETERS_BEGIN(c)
 		KVSM_PARAMETER("directory",KVS_PT_NONEMPTYSTRING,0,szDir)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szDir);
+	if (szDir.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDir);
 
 	QDir d(szDir);
 	if(!d.exists())
@@ -645,12 +660,12 @@ static bool file_kvs_fnc_allSizes(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_fixpath(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
-	c->returnValue()->setString(szName);
+	KviFileUtils::adjustFilePath(szFileName);
+	c->returnValue()->setString(szFileName);
 	return true;
 }
 
@@ -731,7 +746,9 @@ static bool file_kvs_fnc_ls(KviKvsModuleFunctionCall * c)
 		KVSM_PARAMETER("filter",KVS_PT_STRING,KVS_PF_OPTIONAL,szFilter)
 	KVSM_PARAMETERS_END(c)
 
-	KviFileUtils::adjustFilePath(szDir);
+	if (szDir.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szDir);
+
 
 	QDir d(szDir);
 	if(!d.exists())
@@ -823,20 +840,21 @@ static bool file_kvs_fnc_ls(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_read(KviKvsModuleFunctionCall * c)
 {
-	QString szNameZ;
+	QString szFileName;
 	kvs_uint_t uSize;
 	QString szFlags;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szNameZ)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 		KVSM_PARAMETER("size",KVS_PT_UINT,KVS_PF_OPTIONAL,uSize)
 		KVSM_PARAMETER("flags",KVS_PT_STRING,KVS_PF_OPTIONAL,szFlags)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szNameZ);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 
-	QFile f(szNameZ);
+	QFile f(szFileName);
 	if(!f.open(QIODevice::ReadOnly))
 	{
-		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szNameZ);
+		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szFileName);
 		return true;
 	}
 
@@ -853,7 +871,7 @@ static bool file_kvs_fnc_read(KviKvsModuleFunctionCall * c)
 		{
 			// ops
 			KviMemory::free(pcBuf);
-			c->warning(__tr2qs("Read error for file '%Q' (have been unable to read the requested size in 1000 retries)"),&szNameZ);
+			c->warning(__tr2qs("Read error for file '%Q' (have been unable to read the requested size in 1000 retries)"),&szFileName);
 			return true;
 		}
 
@@ -862,7 +880,7 @@ static bool file_kvs_fnc_read(KviKvsModuleFunctionCall * c)
 		if(iReadedNow < 0)
 		{
 			KviMemory::free(pcBuf);
-			c->warning(__tr2qs("Read error for file '%Q'"),&szNameZ);
+			c->warning(__tr2qs("Read error for file '%Q'"),&szFileName);
 			return true;
 		}
 
@@ -915,19 +933,20 @@ static bool file_kvs_fnc_read(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_readBytes(KviKvsModuleFunctionCall * c)
 {
-	QString szNameZ;
+	QString szFileName;
 	kvs_uint_t uSize;
 	QString szFlags;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szNameZ)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 		KVSM_PARAMETER("size",KVS_PT_UINT,KVS_PF_OPTIONAL,uSize)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szNameZ);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 
-	QFile f(szNameZ);
+	QFile f(szFileName);
 	if(!f.open(QIODevice::ReadOnly))
 	{
-		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szNameZ);
+		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szFileName);
 		return true;
 	}
 
@@ -944,7 +963,7 @@ static bool file_kvs_fnc_readBytes(KviKvsModuleFunctionCall * c)
 		{
 			// ops
 			KviMemory::free(pcBuf);
-			c->warning(__tr2qs("Read error for file '%Q' (have been unable to read the requested size in 1000 retries)"),&szNameZ);
+			c->warning(__tr2qs("Read error for file '%Q' (have been unable to read the requested size in 1000 retries)"),&szFileName);
 			return true;
 		}
 
@@ -953,7 +972,7 @@ static bool file_kvs_fnc_readBytes(KviKvsModuleFunctionCall * c)
 		if(iReadedNow < 0)
 		{
 			KviMemory::free(pcBuf);
-			c->warning(__tr2qs("Read error for file '%Q'"),&szNameZ);
+			c->warning(__tr2qs("Read error for file '%Q'"),&szFileName);
 			return true;
 		}
 
@@ -1013,23 +1032,24 @@ static bool file_kvs_fnc_readBytes(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_readLines(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	QString szFlags;
 	kvs_int_t iStartLine;
 	kvs_int_t iCount;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 		KVSM_PARAMETER("startline",KVS_PT_INT,KVS_PF_OPTIONAL,iStartLine)
 		KVSM_PARAMETER("count",KVS_PT_INT,KVS_PF_OPTIONAL,iCount)
 		KVSM_PARAMETER("flags",KVS_PT_STRING,KVS_PF_OPTIONAL,szFlags)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::adjustFilePath(szName);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 
-	QFile f(szName);
+	QFile f(szFileName);
 
 	if(!f.open(QIODevice::ReadOnly))
 	{
-		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szName);
+		c->warning(__tr2qs("Can't open the file '%Q' for reading"),&szFileName);
 		return true;
 	}
 
@@ -1121,16 +1141,17 @@ static bool file_kvs_fnc_readLines(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_cmd_writeLines(KviKvsModuleCommandCall * c)
 {
-	QString szFile,szFlags;
+	QString szFileName,szFlags;
 	KviKvsArrayCast a;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFile)
+		KVSM_PARAMETER("filename",KVS_PT_NONEMPTYSTRING,0,szFileName)
 		KVSM_PARAMETER("lines",KVS_PT_ARRAYCAST,0,a)
 	KVSM_PARAMETERS_END(c)
 
-	KviFileUtils::adjustFilePath(szFile);
+	if (szFileName.left(2)!="\\\\")
+		KviFileUtils::adjustFilePath(szFileName);
 
-	KviFile f(szFile);
+	KviFile f(szFileName);
 
 	bool bAppend = c->switches()->find('a',"append");
 	bool bLocal8Bit = c->switches()->find('l',"local-8-bit");
@@ -1140,7 +1161,7 @@ static bool file_kvs_cmd_writeLines(KviKvsModuleCommandCall * c)
 	if(!f.open(QFile::WriteOnly | (bAppend ? QFile::Append : QFile::Truncate)))
 	{
 		if(!c->switches()->find('q',"quiet"))
-			c->warning(__tr2qs("Can't open the file '%Q' for writing"),&szFile);
+			c->warning(__tr2qs("Can't open the file '%Q' for writing"),&szFileName);
 		return true;
 	}
 
@@ -1201,14 +1222,14 @@ static bool file_kvs_cmd_writeLines(KviKvsModuleCommandCall * c)
 
 static bool file_kvs_fnc_localdir(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szName)
+		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szFileName)
 	KVSM_PARAMETERS_END(c)
-	if(szName.isEmpty())
-		szName.append(KVI_PATH_SEPARATOR_CHAR);
+	if(szFileName.isEmpty())
+		szFileName.append(KVI_PATH_SEPARATOR_CHAR);
 	QString szPath;
-	g_pApp->getLocalKvircDirectory(szPath,KviApplication::None,szName);
+	g_pApp->getLocalKvircDirectory(szPath,KviApplication::None,szFileName);
 	KviFileUtils::adjustFilePath(szPath);
 	c->returnValue()->setString(szPath);
 	return true;
@@ -1235,15 +1256,15 @@ static bool file_kvs_fnc_localdir(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_homedir(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szName)
+		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szFileName)
 	KVSM_PARAMETERS_END(c)
-	if(szName.isEmpty())
-		szName.append(KVI_PATH_SEPARATOR_CHAR);
+	if(szFileName.isEmpty())
+		szFileName.append(KVI_PATH_SEPARATOR_CHAR);
 	QString szPath = QDir::homePath();
 	KviQString::ensureLastCharIs(szPath,KVI_PATH_SEPARATOR_CHAR);
-	szPath.append(szName);
+	szPath.append(szFileName);
 	KviFileUtils::adjustFilePath(szPath);
 	c->returnValue()->setString(szPath);
 	return true;
@@ -1271,15 +1292,15 @@ static bool file_kvs_fnc_homedir(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_rootdir(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szName)
+		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szFileName)
 	KVSM_PARAMETERS_END(c)
-	if(szName.isEmpty())
-		szName.append(KVI_PATH_SEPARATOR_CHAR);
+	if(szFileName.isEmpty())
+		szFileName.append(KVI_PATH_SEPARATOR_CHAR);
 	QString szPath = QDir::rootPath();
 	KviQString::ensureLastCharIs(szPath,KVI_PATH_SEPARATOR_CHAR);
-	szPath.append(szName);
+	szPath.append(szFileName);
 	KviFileUtils::adjustFilePath(szPath);
 	c->returnValue()->setString(szPath);
 	return true;
@@ -1306,15 +1327,15 @@ static bool file_kvs_fnc_rootdir(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_cwd(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szName)
+		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szFileName)
 	KVSM_PARAMETERS_END(c)
-	if(szName.isEmpty())
-		szName.append(KVI_PATH_SEPARATOR_CHAR);
+	if(szFileName.isEmpty())
+		szFileName.append(KVI_PATH_SEPARATOR_CHAR);
 	QString szPath = QDir::currentPath();
 	KviQString::ensureLastCharIs(szPath,KVI_PATH_SEPARATOR_CHAR);
-	szPath.append(szName);
+	szPath.append(szFileName);
 	KviFileUtils::adjustFilePath(szPath);
 	c->returnValue()->setString(szPath);
 	return true;
@@ -1350,14 +1371,14 @@ static bool file_kvs_fnc_cwd(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_globaldir(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szName)
+		KVSM_PARAMETER("relative_path",KVS_PT_STRING,KVS_PF_OPTIONAL,szFileName)
 	KVSM_PARAMETERS_END(c)
-	if(szName.isEmpty())
-		szName.append(KVI_PATH_SEPARATOR_CHAR);
+	if(szFileName.isEmpty())
+		szFileName.append(KVI_PATH_SEPARATOR_CHAR);
 	QString szPath;
-	g_pApp->getGlobalKvircDirectory(szPath,KviApplication::None,szName);
+	g_pApp->getGlobalKvircDirectory(szPath,KviApplication::None,szFileName);
 	KviFileUtils::adjustFilePath(szPath);
 	c->returnValue()->setString(szPath);
 	return true;
@@ -1385,11 +1406,11 @@ static bool file_kvs_fnc_globaldir(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_extractpath(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filepath",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("filepath",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	c->returnValue()->setString(QFileInfo(szName).absolutePath());
+	c->returnValue()->setString(QFileInfo(szFileName).absolutePath());
 	return true;
 }
 
@@ -1414,13 +1435,13 @@ static bool file_kvs_fnc_extractpath(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_extractfilename(KviKvsModuleFunctionCall * c)
 {
-	QString szName;
+	QString szFileName;
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filepath",KVS_PT_NONEMPTYSTRING,0,szName)
+		KVSM_PARAMETER("filepath",KVS_PT_NONEMPTYSTRING,0,szFileName)
 	KVSM_PARAMETERS_END(c)
-	KviFileUtils::extractFileName(szName);
-	KviQString::cutToLast(szName,KVI_PATH_SEPARATOR_CHAR);
-	c->returnValue()->setString(szName);
+	KviFileUtils::extractFileName(szFileName);
+	KviQString::cutToLast(szFileName,KVI_PATH_SEPARATOR_CHAR);
+	c->returnValue()->setString(szFileName);
 	return true;
 }
 
@@ -1598,15 +1619,15 @@ static bool file_kvs_fnc_digest(KviKvsModuleFunctionCall * c)
 
 static bool file_kvs_fnc_time(KviKvsModuleFunctionCall * c)
 {
-	QString szName, szType;
+	QString szFileName, szType;
 
 	KVSM_PARAMETERS_BEGIN(c)
-		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szName)
+		KVSM_PARAMETER("filename",KVS_PT_STRING,0,szFileName)
 		KVSM_PARAMETER("type",KVS_PT_STRING,KVS_PF_OPTIONAL,szType)
 	KVSM_PARAMETERS_END(c)
 
-	KviFileUtils::adjustFilePath(szName);
-	QFileInfo f(szName);
+	KviFileUtils::adjustFilePath(szFileName);
+	QFileInfo f(szFileName);
 
 	QDateTime time;