fixing a few issues with delete and using splice instead

Author: imhotep

lib/pbxProject.js

@@ -86,7 +86,7 @@ pbxProject.prototype.addPluginFile = function (path, opt) {
 
 pbxProject.prototype.removePluginFile = function (path, opt) {
     var file = new pbxFile(path, opt);
-    //correctForPluginsPath(file, this);
+    correctForPluginsPath(file, this);
 
     this.removeFromPbxFileReferenceSection(file);    // PBXFileReference
     this.removeFromPluginsPbxGroup(file);            // PBXGroup
@@ -191,6 +191,7 @@ pbxProject.prototype.removeFromPbxFileReferenceSection = function (file) {
         if(this.pbxFileReferenceSection()[i].name == refObj.name &&
            this.pbxFileReferenceSection()[i].path == refObj.path) {
             file.fileRef = i;
+            delete this.pbxFileReferenceSection()[i];
             break;
         }
     }
@@ -212,7 +213,7 @@ pbxProject.prototype.removeFromPluginsPbxGroup = function (file) {
     for(i in pluginsGroupChildren) {
         if(pbxGroupChild(file).value == pluginsGroupChildren[i].value &&
            pbxGroupChild(file).comment == pluginsGroupChildren[i].comment) {
-            delete pluginsGroupChildren[i];
+            pluginsGroupChildren.splice(i, 1);
             break;
         }
     }
@@ -232,7 +233,7 @@ pbxProject.prototype.removeFromPbxSourcesBuildPhase = function (file) {
     var sources = this.pbxSourcesBuildPhaseObj(), i;
     for(i in sources.files) {
         if(sources.files[i].comment = longComment(file)) {
-            delete sources.files[i];
+            sources.files.splice(i, 1);
             break; 
         }
     }
@@ -248,7 +249,7 @@ pbxProject.prototype.removeFromPbxResourcesBuildPhase = function (file) {
 
     for(i in sources.files) {
         if(sources.files[i].comment = longComment(file)) {
-            delete sources.files[i];
+            sources.files.splice(i, 1);
             break;
         }
     }