Changeset 867 for xmlTools


Ignore:
Timestamp:
May 9, 2013, 12:37:17 PM (12 years ago)
Author:
s10k
Message:

xmlTools: 0.8f nows checks for empty strings and null string to avoid problems like the one fixed on 0.8e

Location:
xmlTools/trunk/posUpdate
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • xmlTools/trunk/posUpdate/ParametersParser.cs

    r861 r867  
    107107            foreach (string currentFile in filesToProcess)
    108108            {
    109                 if (valPositions != null)
     109                if (!String.IsNullOrEmpty(valPositions))
    110110                {
    111111                    myTools.replaceAll(currentFile, value, valPositions);
     
    130130            XmlPatch myPatch;
    131131
    132             if (forceInFiles != "")
     132            if (!String.IsNullOrEmpty(forceInFiles))
    133133            {
    134134                myPatch = new XmlPatch(globalFileName, forceInFiles, globalNoBackups);
     
    196196            List<String> filesToProccess = new List<String>();
    197197
    198             if (filename == "") // No filename? Process everything xml file found.
     198            if (String.IsNullOrEmpty(filename)) // No filename? Process everything xml file found.
    199199            {
    200200                List<string> allXmlFiles = Util.getAllXmlFiles();
  • xmlTools/trunk/posUpdate/Program.cs

    r861 r867  
    1111    class Program
    1212    {
    13         public static readonly string toolsVersion = "0.8e";
     13        public static readonly string toolsVersion = "0.8f";
    1414        private static appErrors lastError = appErrors.NO_ERROR;
    1515
  • xmlTools/trunk/posUpdate/XmlPatch.cs

    r746 r867  
    9696            try
    9797            {
    98                 if (forceFiles == null)
     98                if (String.IsNullOrEmpty(forceFiles))
    9999                {
    100100                    File = getPatchParameter(operation, "File");
     
    115115            }
    116116
    117             if (Element == "")
     117            if (String.IsNullOrEmpty(Element))
    118118            {
    119119                return false;
     
    122122            //---------------------------------------------------Parse Operation command (end)
    123123            List<String> filesToProcess = new List<String>();
    124             if (File == "")
     124            if (String.IsNullOrEmpty(File))
    125125            {
    126126                filesToProcess = Util.getAllXmlFiles(); //no file specified, use all xml files found in same folder
     
    188188            try
    189189            {
    190                 if (forceFiles == null)
     190                if (String.IsNullOrEmpty(forceFiles))
    191191                {
    192192                    File = getPatchParameter(operation, "File");
     
    207207            }
    208208
    209             if (Element == "")
     209            if (String.IsNullOrEmpty(Element))
    210210            {
    211211                return false;
     
    215215
    216216            List<String> filesToProcess = new List<String>();
    217             if (File == "")
     217            if (String.IsNullOrEmpty(File))
    218218            {
    219219                filesToProcess = Util.getAllXmlFiles(); //no file specified, use all xml files found in same folder
     
    272272            command = command.Replace("@COMMAND ", ""); //get only the command to process
    273273
    274             if (command.Trim() == "")
     274            if (String.IsNullOrEmpty(command.Trim()))
    275275            {
    276276                Program.printAppError(Program.appErrors.PATCH_COMMAND_NOT_FOUND, "Error parsing commandOperation in Patch file: Command is empty.");
     
    299299                    }
    300300
    301                     if (paramType != "")
     301                    if (!String.IsNullOrEmpty(paramType))
    302302                    {
    303303                        int startIdx = command.IndexOf(paramType) + paramType.Length;
  • xmlTools/trunk/posUpdate/XmlTools.cs

    r861 r867  
    4141            Util.getAllSpecificElements(xdoc.DocumentElement, ref myElements, this.posElement, this.posParentElement); //Returns all after "Oni" element
    4242
    43             if (valuePositions != "")
     43            if (!String.IsNullOrEmpty(valuePositions))
    4444            {
    4545                checkValidSpecificPositions(valuePositions, value, myElements[0].InnerText);
     
    5050                if (element.Name == posElement)
    5151                {
    52                     if (posParentElement != "" && element.ParentNode.Name != posParentElement)
     52                    if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement)
    5353                    {
    5454                        continue;
    5555                    }
    56                     if (valuePositions == "") //replace every value for the new
     56                    if (String.IsNullOrEmpty(valuePositions)) //replace every value for the new
    5757                    {
    5858                        element.InnerText = value;
     
    9191                if (element.Name == posElement)
    9292                {
    93                     if (posParentElement != "" && element.ParentNode.Name != posParentElement)
     93                    if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement)
    9494                    {
    9595                        continue;
     
    141141                if (element.Name == posElement)
    142142                {
    143                     if (posParentElement != "" && element.ParentNode.Name != posParentElement)
     143                    if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement)
    144144                    {
    145145                        continue;
     
    185185                if (element.Name == posElement)
    186186                {
    187                     if (posParentElement != "" && element.ParentNode.Name != posParentElement)
     187                    if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement)
    188188                    {
    189189                        continue;
     
    230230            Util.getAllSpecificElements(xdoc.DocumentElement, ref myElements, this.posElement, this.posParentElement); //Returns all after "Oni" element
    231231
    232             if (valuePositions == null)
     232            if (String.IsNullOrEmpty(valuePositions))
    233233            {
    234234                newXmlLastPos = new XmlNumberValue(newValue);
     
    239239            }
    240240
    241             if (valuePositions != null)
     241            if (!String.IsNullOrEmpty(valuePositions))
    242242            {
    243243                checkValidSpecificPositions(valuePositions, newValue, myElements[0].InnerText);
     
    246246            foreach (XmlNode element in myElements)
    247247            {
    248                 if (element.Name == this.posElement && (this.posParentElement == "" || this.posParentElement == element.ParentNode.Name))
     248                if (element.Name == this.posElement && (String.IsNullOrEmpty(this.posParentElement) || this.posParentElement == element.ParentNode.Name))
    249249                {
    250250                    XmlNumberValue xmlCurrValue = new XmlNumberValue(element.InnerText);
    251251
    252                     if (xmlLastPos != null)
     252                    if (xmlLastPos!=null)
    253253                    {
    254254                        newXmlLastPos = XmlNumberValue.sum(newXmlLastPos, XmlNumberValue.difference(xmlCurrValue, xmlLastPos));
     
    258258                    else
    259259                    { // first time just assign to last value
    260                         if (valueRelation != null)
     260                        if (!String.IsNullOrEmpty(valueRelation))
    261261                        {
    262262                            newXmlLastPos = XmlNumberValue.difference(newXmlLastPos, XmlNumberValue.difference(new XmlNumberValue(valueRelation), xmlCurrValue));
Note: See TracChangeset for help on using the changeset viewer.