Fixes another merge issue and improves performance slightly for sorting so that we are not double saving the entity if it is published already.

src/Umbraco.Web/umbraco.presentation/umbraco/webservices/nodeSorter.asmx.cs

@@ -108,6 +108,11 @@ namespace umbraco.presentation.webservices
                             //TODO: WE don't want to have to republish the entire document !!!!
                             library.UpdateDocumentCache(document);
                         }
+                        else
+                        {
+                            //we need to save it if it is not published to persist the sort order value
+                            document.Save();
+                        }
                     }
                         // to update the sortorder of the media node in the XML, re-save the node....
                     else if (isMedia)
@@ -116,10 +121,7 @@ namespace umbraco.presentation.webservices
                             {
                                 sortOrder = i
                             };
-                        media.Save();
-                        {
-                            new cms.businesslogic.CMSNode(int.Parse(ids[i])).sortOrder = i;
-                        }
+                        media.Save();                        
                     }
 
                     // Refresh sort order on cached xml