При смене fontName добавляем в историю сами (а не на фунции). Баг...

При смене fontName добавляем в историю сами (а не на фунции). Баг http://bugzserver/show_bug.cgi?id=25200 Проблема была в том, что в истории не было еще точек, а мы уже добавляли.
ToDo стоит парсить уже на DeserializeHistory имена шрифтов.


git-svn-id: svn://192.168.3.15/activex/AVS/Sources/TeamlabOffice/trunk/OfficeWeb@57134 954022d7-b5bf-4e40-9824-e11837661b57

Common/Drawings/CommonController.js

@@ -1215,9 +1215,14 @@ DrawingObjectsController.prototype =
     },
 
     setCellFontName: function (fontName) {
+		var t = this;
         var text_pr = new ParaTextPr({ FontFamily : { Name : fontName , Index : -1 } });
-        this.checkSelectedObjectsAndCallback(this.paragraphAdd, [text_pr]);
-
+        this.checkSelectedObjectsAndCallback(function () {
+			// ToDo стоит перейти на проверку шрифтов на deserialize
+			History.Add(g_oUndoRedoWorkbook, historyitem_Workbook_AddFont, null, null,
+				new UndoRedoData_SingleProperty([fontName]), true);
+			t.paragraphAdd(text_pr);
+		}, []);
     },
 
     setCellFontSize: function (fontSize) {

Excel/api.js

@@ -1324,10 +1324,6 @@ var ASC_DOCS_API_USE_EMBEDDED_FONTS = "@@ASC_DOCS_API_USE_EMBEDDED_FONTS";
 			_loadFonts: function (fonts, callback) {
 			    if (window["NATIVE_EDITOR_ENJINE"])
 			        return callback();
-			    var aFonts = [];
-			    for (var i in fonts)
-			        aFonts.push(i);
-			    History.Add(g_oUndoRedoWorkbook, historyitem_Workbook_AddFont, null, null, new UndoRedoData_SingleProperty(aFonts), true);
 				this.asyncMethodCallback = callback;
 				var arrLoadFonts = [];
 				for(var i in fonts)

Excel/model/UndoRedo.js

@@ -212,7 +212,6 @@ UndoRedoItemSerializable.prototype = {
 	},
 
 
-
 	Deserialize : function(oBinaryReader)
 	{
 		var res = c_oSerConstants.ReadOk;
@@ -258,10 +257,9 @@ UndoRedoItemSerializable.prototype = {
 	},
 	DeserializeData : function(oBinaryReader)
 	{
-		var oDataObject = null;
 		var nDataClassType = oBinaryReader.GetUChar();
 		var nLength = oBinaryReader.GetULongLE();
-		oDataObject = UndoRedoDataTypes.Create(nDataClassType);
+		var oDataObject = UndoRedoDataTypes.Create(nDataClassType);
 		if(null != oDataObject)
 		{
 			if(null != oDataObject.Read_FromBinary2)

Excel/model/Workbook.js

@@ -1768,7 +1768,7 @@ Workbook.prototype._SerializeHistoryBase64 = function (oMemory, item, aPointChan
         if (nLen > 0)
             aPointChangesBase64.push(nLen + ";" + oMemory.GetBase64Memory2(nPosStart, nLen));
     }
-}
+};
 Workbook.prototype.SerializeHistory = function(){
 	var aRes = [];
 	//соединяем изменения, которые были до приема данных с теми, что получились после.

Excel/model/clipboard.js

@@ -1665,6 +1665,7 @@
                 targetContent.Remove(1, true, true);
 					
 				worksheet._loadFonts(newFonts, function () {
+					worksheet._addFontsToHistory(newFonts);
 					oPasteProcessor.InsertInPlace(targetContent , oPasteProcessor.aContent);
 					
 					worksheet.objectRender.controller.startRecalculate();

Excel/view/WorksheetView.js

@@ -7854,7 +7854,13 @@
 				History.StartTransaction();
 
 				switch (prop) {
-					case "fn": range.setFontname(val); canChangeColWidth = c_oAscCanChangeColWidth.numbers; break;
+					case "fn":
+						// ToDo стоит перейти на проверку шрифтов на deserialize
+						History.Add(g_oUndoRedoWorkbook, historyitem_Workbook_AddFont, null, null,
+							new UndoRedoData_SingleProperty([val]), true);
+						range.setFontname(val);
+						canChangeColWidth = c_oAscCanChangeColWidth.numbers;
+						break;
 					case "fs": range.setFontsize(val); canChangeColWidth = c_oAscCanChangeColWidth.numbers; break;
 					case "b":  range.setBold(val); break;
 					case "i":  range.setItalic(val); break;
@@ -8125,6 +8131,8 @@
 			var t = this;
 			//загрузка шрифтов, в случае удачи на callback вставляем текст
 			t._loadFonts(val.fontsNew, function () {
+				t._addFontsToHistory(val.fontsNew);
+
 				if(val.onlyImages !== true)
 					t._pasteFromLocalBuff(isLargeRange, isLocal, val, bIsUpdate, canChangeColWidth);
 					
@@ -8742,6 +8750,10 @@
 			var arrFormula = [];
 			var numFor = 0;
 
+			// ToDo не совсем грамотная правка, но пока так
+			var newFonts = {};
+			this._addFontsToHistory(val.generateFontMap(newFonts));
+
 			var pasteRange = window["Asc"]["editor"].wb.clipboard.activeRange;
 			var activeCellsPasteFragment = this.autoFilters._refToRange(pasteRange);
 			var rMax = (activeCellsPasteFragment.r2 - activeCellsPasteFragment.r1) + arn.r1 + 1;
@@ -10707,6 +10719,15 @@
 			api._loadFonts(fonts, callback);
 		};
 
+		WorksheetView.prototype._addFontsToHistory = function (fonts) {
+			var arrNewFonts = [];
+			for (var oFontName in fonts)
+				arrNewFonts.push(oFontName);
+			// ToDo стоит перейти на проверку шрифтов на deserialize
+			History.Add(g_oUndoRedoWorkbook, historyitem_Workbook_AddFont, null, null,
+				new UndoRedoData_SingleProperty(arrNewFonts), true);
+		};
+
 		/*
 		 * Export
 		 * -----------------------------------------------------------------------------