From 41317cab7101f0f4fa438563373e1441b6ff3f70 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Fri, 11 Feb 2022 21:59:07 +0300
Subject: [PATCH 1/9] Fix missing exact sku-group search and fill.

---
 src/PriceListTools/AbstractTool.cs | 73 +++++++++++++++---------------
 1 file changed, 36 insertions(+), 37 deletions(-)

diff --git a/src/PriceListTools/AbstractTool.cs b/src/PriceListTools/AbstractTool.cs
index d312315..68c5250 100644
--- a/src/PriceListTools/AbstractTool.cs
+++ b/src/PriceListTools/AbstractTool.cs
@@ -58,7 +58,7 @@ namespace RehauSku.PriceListTools
 
         protected private void FillPositionAmountToColumns(KeyValuePair<Position, double> positionAmount, params int[] columns)
         {
-            int? row = GetPositionRow(positionAmount.Key.Sku, positionAmount.Key.Group, TargetFile.skuCell.Column);
+            int? row = GetPositionRow(TargetFile.skuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
 
             if (row != null)
             {
@@ -69,11 +69,12 @@ namespace RehauSku.PriceListTools
                 }
 
                 ResultBar.IncrementSuccess();
+                return;
             }
 
-            else if (TargetFile.oldSkuCell != null)
+            if (TargetFile.oldSkuCell != null)
             {
-                row = GetPositionRow(positionAmount.Key.Sku, positionAmount.Key.Group, TargetFile.oldSkuCell.Column);
+                row = GetPositionRow(TargetFile.oldSkuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
 
                 if (row != null)
                 {
@@ -84,31 +85,27 @@ namespace RehauSku.PriceListTools
                     }
 
                     ResultBar.IncrementReplaced();
+                    return;
                 }
             }
 
-            else
+            string sku = positionAmount.Key.Sku.Substring(1, 6);
+            row = GetPositionRow(TargetFile.skuCell.EntireColumn, sku, positionAmount.Key.Group);
+
+            if (row != null)
             {
-                string sku = positionAmount.Key.Sku.Substring(1, 6);
-                row = GetPositionRow(sku, positionAmount.Key.Group, TargetFile.skuCell.Column);
-
-                if (row != null)
+                foreach (int column in columns)
                 {
-                    foreach (int column in columns)
-                    {
-                        Range cell = TargetFile.Sheet.Cells[row, column];
-                        cell.AddValue(positionAmount.Value);
-                    }
-
-                    ResultBar.IncrementReplaced();
+                    Range cell = TargetFile.Sheet.Cells[row, column];
+                    cell.AddValue(positionAmount.Value);
                 }
 
-                else
-                {
-                    FillMissing(positionAmount, columns);
-                    ResultBar.IncrementNotFound();
-                }
+                ResultBar.IncrementReplaced();
+                return;
             }
+
+            FillMissing(positionAmount, columns);
+            ResultBar.IncrementNotFound();
         }
 
         protected private void FillMissing(KeyValuePair<Position, double> positionAmount, params int[] columns)
@@ -148,32 +145,34 @@ namespace RehauSku.PriceListTools
             }
         }
 
-        protected private int? GetPositionRow(string sku, string group, int column)
+        protected private int? GetPositionRow(Range range, string sku, string group)
         {
-            int? row = null;
-            Range foundCell = TargetFile.Sheet.Columns[column].Find(sku);
+            Range found = range.Find(sku);
             string foundGroupValue;
 
-            if (foundCell == null) return null;
-
-            else
+            if (found == null)
             {
-                row = foundCell.Row;
-                foundGroupValue = TargetFile.Sheet.Cells[foundCell.Row, TargetFile.groupCell.Column].Value2.ToString();
+                return null;
             }
 
-            if (string.IsNullOrEmpty(group) || group.Equals(foundGroupValue))
-                return row;
+            int firstFoundRow = found.Row; 
 
-            else
-                while (true)
+            while (true)
+            {
+                foundGroupValue = TargetFile.Sheet.Cells[found.Row, TargetFile.groupCell.Column].Value2.ToString();
+
+                if (string.IsNullOrEmpty(group) || group.Equals(foundGroupValue))
                 {
-                    foundCell = TargetFile.skuCell.EntireColumn.FindNext(foundCell);
-                    if (foundCell == null) return row;
-
-                    foundGroupValue = TargetFile.Sheet.Cells[foundCell.Row, TargetFile.groupCell.Column].Value2.ToString();
-                    if (group.Equals(foundGroupValue)) return foundCell.Row;
+                    return found.Row;
                 }
+
+                found = range.FindNext(found);
+
+                if (found.Row == firstFoundRow)
+                {
+                    return null;
+                }
+            }
         }
 
         protected private void FilterByAmount()

From 69865e96e3a1f128c9c25a32a445745aa8aa4fbf Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Sat, 12 Feb 2022 16:02:55 +0300
Subject: [PATCH 2/9] Create Tool Factory. Exception on empty selection in
 Export Tool.

---
 src/Interface/RibbonController.cs  | 98 ++++++++++++------------------
 src/PriceListTools/AbstractTool.cs | 31 +++-------
 src/PriceListTools/CombineTool.cs  | 23 ++++++-
 src/PriceListTools/ConvertTool.cs  | 21 ++-----
 src/PriceListTools/ExportTool.cs   |  9 ++-
 src/PriceListTools/MergeTool.cs    | 20 +++++-
 6 files changed, 99 insertions(+), 103 deletions(-)

diff --git a/src/Interface/RibbonController.cs b/src/Interface/RibbonController.cs
index bc038d1..7a58c7c 100644
--- a/src/Interface/RibbonController.cs
+++ b/src/Interface/RibbonController.cs
@@ -20,11 +20,11 @@ namespace RehauSku.Interface
         <tabs>
           <tab id='rau' label='REHAU'>
             <group id='priceList' label='Прайс-лист'>
-                <button id='exportToPrice' getEnabled='GetExportEnabled' label='Экспорт в новый файл' size='normal' imageMso='PivotExportToExcel' onAction='OnExportPressed'/> 
-                <button id='convertPrice' getEnabled='GetConvertEnabled' label='Актуализировать' size='normal' imageMso='FileUpdate' onAction='OnConvertPressed'/> 
+                <button id='exportToPrice' getEnabled='GetExportEnabled' label='Экспорт в новый файл' size='normal' imageMso='PivotExportToExcel' onAction='OnToolPressed'/> 
+                <button id='convertPrice' getEnabled='GetConvertEnabled' label='Актуализировать' size='normal' imageMso='FileUpdate' onAction='OnToolPressed'/> 
                 <menu id='conjoinMenu' label='Объединить' imageMso='Copy'>
-                    <button id='mergeFiles' label='Сложить' onAction='OnMergePressed'/>    
-                    <button id='combineFiles' label='По колонкам' onAction='OnCombinePressed'/>   
+                    <button id='mergeFiles' label='Сложить' onAction='OnToolPressed'/>    
+                    <button id='combineFiles' label='По колонкам' onAction='OnToolPressed'/>   
                 </menu>
             </group>
             <group id='rausettings' label='Настройки'>
@@ -48,30 +48,50 @@ namespace RehauSku.Interface
                 ribbonUi.InvalidateControl(id);
             }
         }
-
-        public void OnMergePressed(IRibbonControl control)
+        public void OnSetPricePressed(IRibbonControl control)
         {
-            MergeTool mergeTool = new MergeTool();
-            string[] files = Dialog.GetMultiplyFiles();
+            string path = Dialog.GetFilePath();
 
-            if (files != null)
+            if (!string.IsNullOrEmpty(path))
             {
-                mergeTool.SourceFiles = SourcePriceList.GetSourceLists(files);
-                mergeTool.OpenNewPrice();
-                mergeTool.FillTarget();
+                RegistryUtil.PriceListPath = path;
             }
         }
 
-        public void OnCombinePressed(IRibbonControl control)
+        public void OnToolPressed(IRibbonControl control)
         {
-            CombineTool combineTool = new CombineTool();
-            string[] files = Dialog.GetMultiplyFiles();
-
-            if (files != null)
+            try
             {
-                combineTool.SourceFiles = SourcePriceList.GetSourceLists(files);
-                combineTool.OpenNewPrice();
-                combineTool.FillTarget();
+                AbstractTool tool;
+                switch (control.Id)
+                {
+                    case "exportToPrice":
+                        tool = new ExportTool();
+                        break;
+                    case "convertPrice":
+                        tool = new ConvertTool();
+                        break;
+                    case "mergeFiles":
+                        tool = new MergeTool();
+                        break;
+                    case "combineFiles":
+                        tool = new CombineTool();
+                        break;
+                    default:
+                        throw new Exception("Неизвестный инструмент");
+                }
+
+                tool.OpenNewPrice();
+                tool.FillTarget();
+            }
+
+            catch (Exception exception)
+            {
+                MessageBox.Show(exception.Message,
+                    "Ошибка",
+                    MessageBoxButtons.OK,
+                    MessageBoxIcon.Information);
+                return;
             }
         }
 
@@ -87,25 +107,6 @@ namespace RehauSku.Interface
             }
         }
 
-        public void OnExportPressed(IRibbonControl control)
-        {
-            try
-            {
-                ExportTool exportTool = new ExportTool();
-                exportTool.OpenNewPrice();
-                exportTool.FillTarget();
-            }
-
-            catch (Exception ex)
-            {
-                MessageBox.Show(ex.Message,
-                    "Ошибка",
-                    MessageBoxButtons.OK,
-                    MessageBoxIcon.Information);
-                return;
-            }
-        }
-
         public bool GetExportEnabled(IRibbonControl control)
         {
             if (AddIn.Excel.ActiveWorkbook == null)
@@ -117,24 +118,5 @@ namespace RehauSku.Interface
                 return selection.Columns.Count == 2;
             }
         }
-
-        public void OnConvertPressed(IRibbonControl control)
-        {
-            ConvertTool convertTool = new ConvertTool();
-
-            convertTool.GetCurrent();
-            convertTool.OpenNewPrice();
-            convertTool.FillTarget();
-        }
-
-        public void OnSetPricePressed(IRibbonControl control)
-        {
-            string path = Dialog.GetFilePath();
-
-            if (!string.IsNullOrEmpty(path))
-            {
-                RegistryUtil.PriceListPath = path;
-            }
-        }
     }
 }
diff --git a/src/PriceListTools/AbstractTool.cs b/src/PriceListTools/AbstractTool.cs
index 68c5250..af60d0d 100644
--- a/src/PriceListTools/AbstractTool.cs
+++ b/src/PriceListTools/AbstractTool.cs
@@ -4,7 +4,6 @@ using RehauSku.Interface;
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using System.Windows.Forms;
 using Application = Microsoft.Office.Interop.Excel.Application;
 using ProgressBar = RehauSku.Interface.ProgressBar;
 
@@ -12,10 +11,12 @@ namespace RehauSku.PriceListTools
 {
     internal abstract class AbstractTool
     {
-        protected private Application ExcelApp = (Application)ExcelDnaUtil.Application;
-        protected private TargetPriceList TargetFile;
-        protected private ResultBar ResultBar { get; set; }
-        protected private ProgressBar ProgressBar { get; set; }
+        protected Application ExcelApp = (Application)ExcelDnaUtil.Application;
+        protected TargetPriceList TargetFile { get; set; }
+        protected ResultBar ResultBar { get; set; }
+        protected ProgressBar ProgressBar { get; set; }
+
+        public abstract void FillTarget();
 
         public void OpenNewPrice()
         {
@@ -23,12 +24,6 @@ namespace RehauSku.PriceListTools
                 .Cast<Workbook>()
                 .FirstOrDefault(w => w.FullName == RegistryUtil.PriceListPath) != null)
             {
-                MessageBox.Show
-                    ("Шаблонный файл редактируется в другом месте",
-                    "Ошибка открытия шаблонного прайс-листа",
-                    MessageBoxButtons.OK,
-                    MessageBoxIcon.Information);
-
                 throw new ArgumentException("Шаблонный файл редактируется в другом месте");
             }
 
@@ -41,12 +36,6 @@ namespace RehauSku.PriceListTools
 
             catch (Exception exception)
             {
-                MessageBox.Show
-                    (exception.Message,
-                    "Ошибка открытия шаблонного прайс-листа",
-                    MessageBoxButtons.OK,
-                    MessageBoxIcon.Information);
-
                 if (wb != null)
                 {
                     wb.Close();
@@ -56,7 +45,7 @@ namespace RehauSku.PriceListTools
             }
         }
 
-        protected private void FillPositionAmountToColumns(KeyValuePair<Position, double> positionAmount, params int[] columns)
+        protected void FillPositionAmountToColumns(KeyValuePair<Position, double> positionAmount, params int[] columns)
         {
             int? row = GetPositionRow(TargetFile.skuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
 
@@ -108,7 +97,7 @@ namespace RehauSku.PriceListTools
             ResultBar.IncrementNotFound();
         }
 
-        protected private void FillMissing(KeyValuePair<Position, double> positionAmount, params int[] columns)
+        protected void FillMissing(KeyValuePair<Position, double> positionAmount, params int[] columns)
         {
             int row = TargetFile.Sheet.Cells[TargetFile.Sheet.Rows.Count, TargetFile.skuCell.Column]
                 .End[XlDirection.xlUp]
@@ -145,7 +134,7 @@ namespace RehauSku.PriceListTools
             }
         }
 
-        protected private int? GetPositionRow(Range range, string sku, string group)
+        protected int? GetPositionRow(Range range, string sku, string group)
         {
             Range found = range.Find(sku);
             string foundGroupValue;
@@ -175,7 +164,7 @@ namespace RehauSku.PriceListTools
             }
         }
 
-        protected private void FilterByAmount()
+        protected void FilterByAmount()
         {
             AutoFilter filter = TargetFile.Sheet.AutoFilter;
             int startColumn = filter.Range.Column;
diff --git a/src/PriceListTools/CombineTool.cs b/src/PriceListTools/CombineTool.cs
index af9378c..e3cb83e 100644
--- a/src/PriceListTools/CombineTool.cs
+++ b/src/PriceListTools/CombineTool.cs
@@ -1,15 +1,32 @@
 using Microsoft.Office.Interop.Excel;
-using System.Collections.Generic;
 using RehauSku.Interface;
+using System;
+using System.Collections.Generic;
 using System.Linq;
+using Dialog = RehauSku.Interface.Dialog;
 
 namespace RehauSku.PriceListTools
 {
     internal class CombineTool : AbstractTool
     {
-        public List<SourcePriceList> SourceFiles;
+        private List<SourcePriceList> SourceFiles { get; set; }
 
-        public void FillTarget()
+        public CombineTool()
+        {
+            string[] files = Dialog.GetMultiplyFiles();
+
+            if (files != null)
+            {
+                SourceFiles = SourcePriceList.GetSourceLists(files);
+            }
+
+            else
+            {
+                throw new Exception("Не выбраны файлы");
+            }
+        }
+
+        public override void FillTarget()
         {
             ProgressBar = new ProgressBar("Заполняю строки...", SourceFiles.Sum(file => file.PositionAmount.Count));
             ResultBar = new ResultBar();
diff --git a/src/PriceListTools/ConvertTool.cs b/src/PriceListTools/ConvertTool.cs
index d13c803..fc5850f 100644
--- a/src/PriceListTools/ConvertTool.cs
+++ b/src/PriceListTools/ConvertTool.cs
@@ -5,27 +5,14 @@ namespace RehauSku.PriceListTools
 {
     internal class ConvertTool : AbstractTool
     {
-        private SourcePriceList Current;
+        private SourcePriceList Current { get; set; }
 
-        public void GetCurrent()
+        public ConvertTool()
         {
-            try
-            {
-                Current = new SourcePriceList(ExcelApp.ActiveWorkbook);
-            }
-
-            catch (Exception exception)
-            {
-                System.Windows.Forms.MessageBox.Show
-                    (exception.Message,
-                    "Ошибка распознавания",
-                    System.Windows.Forms.MessageBoxButtons.OK,
-                    System.Windows.Forms.MessageBoxIcon.Information);
-                throw exception;
-            }
+            Current = new SourcePriceList(ExcelApp.ActiveWorkbook);
         }
 
-        public void FillTarget()
+        public override void FillTarget()
         {
             ProgressBar = new ProgressBar("Заполняю строки...", Current.PositionAmount.Count);
             ResultBar = new ResultBar();
diff --git a/src/PriceListTools/ExportTool.cs b/src/PriceListTools/ExportTool.cs
index f341671..acc6515 100644
--- a/src/PriceListTools/ExportTool.cs
+++ b/src/PriceListTools/ExportTool.cs
@@ -13,11 +13,16 @@ namespace RehauSku.PriceListTools
         public ExportTool()
         {
             Selection = ExcelApp.Selection;
+            GetSelected();
+
+            if (PositionAmount.Count == 0)
+            {
+                throw new Exception("В выделенном диапазоне не найдены позиции для экспорта");
+            }
         }
 
-        public void FillTarget()
+        public override void FillTarget()
         {
-            GetSelected();
             ProgressBar = new ProgressBar("Заполняю строки...", PositionAmount.Count);
             ResultBar = new ResultBar();
             
diff --git a/src/PriceListTools/MergeTool.cs b/src/PriceListTools/MergeTool.cs
index 0e3f1dc..77732bf 100644
--- a/src/PriceListTools/MergeTool.cs
+++ b/src/PriceListTools/MergeTool.cs
@@ -1,4 +1,5 @@
 using RehauSku.Interface;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 
@@ -6,9 +7,24 @@ namespace RehauSku.PriceListTools
 {
     internal class MergeTool : AbstractTool
     {
-        public List<SourcePriceList> SourceFiles;
+        private List<SourcePriceList> SourceFiles { get; set; }
 
-        public void FillTarget()
+        public MergeTool()
+        {
+            string[] files = Dialog.GetMultiplyFiles();
+
+            if (files != null)
+            {
+                SourceFiles = SourcePriceList.GetSourceLists(files);
+            }
+
+            else
+            {
+                throw new Exception("Не выбраны файлы");
+            }
+        }
+
+        public override void FillTarget()
         {
             ProgressBar = new ProgressBar("Заполняю строки...", SourceFiles.Sum(x => x.PositionAmount.Count));
             ResultBar = new ResultBar();

From 3224a8967e4b2252f25d12dcaaa7d2c4acc87875 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Sat, 12 Feb 2022 16:08:01 +0300
Subject: [PATCH 3/9] Refactoring

---
 src/PriceListTools/AbstractTool.cs | 6 ++----
 src/PriceListTools/ConvertTool.cs  | 1 -
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/PriceListTools/AbstractTool.cs b/src/PriceListTools/AbstractTool.cs
index af60d0d..c43d06c 100644
--- a/src/PriceListTools/AbstractTool.cs
+++ b/src/PriceListTools/AbstractTool.cs
@@ -1,17 +1,15 @@
-using ExcelDna.Integration;
-using Microsoft.Office.Interop.Excel;
+using Microsoft.Office.Interop.Excel;
 using RehauSku.Interface;
 using System;
 using System.Collections.Generic;
 using System.Linq;
-using Application = Microsoft.Office.Interop.Excel.Application;
 using ProgressBar = RehauSku.Interface.ProgressBar;
 
 namespace RehauSku.PriceListTools
 {
     internal abstract class AbstractTool
     {
-        protected Application ExcelApp = (Application)ExcelDnaUtil.Application;
+        protected Application ExcelApp = AddIn.Excel;
         protected TargetPriceList TargetFile { get; set; }
         protected ResultBar ResultBar { get; set; }
         protected ProgressBar ProgressBar { get; set; }
diff --git a/src/PriceListTools/ConvertTool.cs b/src/PriceListTools/ConvertTool.cs
index fc5850f..abf7e39 100644
--- a/src/PriceListTools/ConvertTool.cs
+++ b/src/PriceListTools/ConvertTool.cs
@@ -1,5 +1,4 @@
 using RehauSku.Interface;
-using System;
 
 namespace RehauSku.PriceListTools
 {

From 3feab7cf5f8755b4c05a6d57aece037947b16273 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Sat, 12 Feb 2022 16:53:34 +0300
Subject: [PATCH 4/9] Extract PriceListHeaders class

---
 src/AddIn/EventsUtil.cs                 |  2 +-
 src/AddIn/WorksheetExtensions.cs        | 14 +++++--------
 src/Interface/ProgressBar.cs            |  2 +-
 src/PriceListTools/AbstractPriceList.cs | 13 ++++---------
 src/PriceListTools/AbstractTool.cs      | 26 ++++++++++++-------------
 src/PriceListTools/CombineTool.cs       |  6 +++---
 src/PriceListTools/ConvertTool.cs       |  2 +-
 src/PriceListTools/ExportTool.cs        |  4 ++--
 src/PriceListTools/MergeTool.cs         |  2 +-
 src/PriceListTools/PriceListHeaders.cs  | 11 +++++++++++
 src/PriceListTools/SourcePriceList.cs   | 19 +++++++++---------
 src/PriceListTools/TargetPriceList.cs   | 13 ++++++-------
 src/RehauSku.Assist.csproj              |  1 +
 13 files changed, 58 insertions(+), 57 deletions(-)
 create mode 100644 src/PriceListTools/PriceListHeaders.cs

diff --git a/src/AddIn/EventsUtil.cs b/src/AddIn/EventsUtil.cs
index 102e12e..bc8d1d3 100644
--- a/src/AddIn/EventsUtil.cs
+++ b/src/AddIn/EventsUtil.cs
@@ -4,7 +4,7 @@ namespace RehauSku
 {
     internal static class EventsUtil
     {
-        private static Application Excel = AddIn.Excel;
+        private static readonly Application Excel = AddIn.Excel;
 
         public static void Initialize()
         {
diff --git a/src/AddIn/WorksheetExtensions.cs b/src/AddIn/WorksheetExtensions.cs
index cffa55c..7880b66 100644
--- a/src/AddIn/WorksheetExtensions.cs
+++ b/src/AddIn/WorksheetExtensions.cs
@@ -1,15 +1,11 @@
 using Microsoft.Office.Interop.Excel;
+using RehauSku.PriceListTools;
 using System.Linq;
 
 namespace RehauSku
 {
     public static class WorksheetExtensions
     {
-        private static string amountHeader = "Кол-во";
-        private static string skuHeader = "Актуальный материал";
-        private static string groupHeader = "Программа";
-        private static string nameHeader = "Наименование";
-
         public static bool IsRehauSource(this Worksheet worksheet)
         {
             Range amountCell;
@@ -19,10 +15,10 @@ namespace RehauSku
 
             Range[] cells = new[]
             {
-                amountCell = worksheet.Cells.Find(amountHeader),
-                skuCell = worksheet.Cells.Find(skuHeader),
-                groupCell = worksheet.Cells.Find(groupHeader),
-                nameCell = worksheet.Cells.Find(nameHeader)
+                amountCell = worksheet.Cells.Find(PriceListHeaders.Amount),
+                skuCell = worksheet.Cells.Find(PriceListHeaders.Sku),
+                groupCell = worksheet.Cells.Find(PriceListHeaders.Group),
+                nameCell = worksheet.Cells.Find(PriceListHeaders.Name)
             };
 
             return cells.All(x => x != null);
diff --git a/src/Interface/ProgressBar.cs b/src/Interface/ProgressBar.cs
index 2e68e8b..416c7d6 100644
--- a/src/Interface/ProgressBar.cs
+++ b/src/Interface/ProgressBar.cs
@@ -19,7 +19,7 @@
 
             if (percent < 100)
             {
-                Excel.StatusBar = $"{Message} Выполнено {percent.ToString("#.#")} %";
+                Excel.StatusBar = $"{Message} Выполнено {percent:#.#} %";
             }
 
             else
diff --git a/src/PriceListTools/AbstractPriceList.cs b/src/PriceListTools/AbstractPriceList.cs
index 06427a0..2f89a85 100644
--- a/src/PriceListTools/AbstractPriceList.cs
+++ b/src/PriceListTools/AbstractPriceList.cs
@@ -4,15 +4,10 @@ namespace RehauSku.PriceListTools
 {
     internal abstract class AbstractPriceList
     {
-        protected const string amountHeader = "Кол-во";
-        protected const string skuHeader = "Актуальный материал";
-        protected const string groupHeader = "Программа";
-        protected const string nameHeader = "Наименование";
-
-        public Range amountCell { get; protected set; }
-        public Range skuCell { get; protected set; }
-        public Range groupCell { get; protected set; }
-        public Range nameCell { get; protected set; }
+        public Range AmountCell { get; protected set; }
+        public Range SkuCell { get; protected set; }
+        public Range GroupCell { get; protected set; }
+        public Range NameCell { get; protected set; }
 
         public Worksheet Sheet { get; protected set; }
         public string Name { get; protected set; }
diff --git a/src/PriceListTools/AbstractTool.cs b/src/PriceListTools/AbstractTool.cs
index c43d06c..256be06 100644
--- a/src/PriceListTools/AbstractTool.cs
+++ b/src/PriceListTools/AbstractTool.cs
@@ -45,7 +45,7 @@ namespace RehauSku.PriceListTools
 
         protected void FillPositionAmountToColumns(KeyValuePair<Position, double> positionAmount, params int[] columns)
         {
-            int? row = GetPositionRow(TargetFile.skuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
+            int? row = GetPositionRow(TargetFile.SkuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
 
             if (row != null)
             {
@@ -59,9 +59,9 @@ namespace RehauSku.PriceListTools
                 return;
             }
 
-            if (TargetFile.oldSkuCell != null)
+            if (TargetFile.OldSkuCell != null)
             {
-                row = GetPositionRow(TargetFile.oldSkuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
+                row = GetPositionRow(TargetFile.OldSkuCell.EntireColumn, positionAmount.Key.Sku, positionAmount.Key.Group);
 
                 if (row != null)
                 {
@@ -77,7 +77,7 @@ namespace RehauSku.PriceListTools
             }
 
             string sku = positionAmount.Key.Sku.Substring(1, 6);
-            row = GetPositionRow(TargetFile.skuCell.EntireColumn, sku, positionAmount.Key.Group);
+            row = GetPositionRow(TargetFile.SkuCell.EntireColumn, sku, positionAmount.Key.Group);
 
             if (row != null)
             {
@@ -97,7 +97,7 @@ namespace RehauSku.PriceListTools
 
         protected void FillMissing(KeyValuePair<Position, double> positionAmount, params int[] columns)
         {
-            int row = TargetFile.Sheet.Cells[TargetFile.Sheet.Rows.Count, TargetFile.skuCell.Column]
+            int row = TargetFile.Sheet.Cells[TargetFile.Sheet.Rows.Count, TargetFile.SkuCell.Column]
                 .End[XlDirection.xlUp]
                 .Row + 1;
 
@@ -111,18 +111,18 @@ namespace RehauSku.PriceListTools
             previous.Copy(current);
             current.ClearContents();
 
-            TargetFile.Sheet.Cells[row, TargetFile.groupCell.Column].Value2 = positionAmount.Key.Group;
-            TargetFile.Sheet.Cells[row, TargetFile.nameCell.Column].Value2 = positionAmount.Key.Name;
+            TargetFile.Sheet.Cells[row, TargetFile.GroupCell.Column].Value2 = positionAmount.Key.Group;
+            TargetFile.Sheet.Cells[row, TargetFile.NameCell.Column].Value2 = positionAmount.Key.Name;
 
-            if (TargetFile.oldSkuCell != null)
+            if (TargetFile.OldSkuCell != null)
             {
-                TargetFile.Sheet.Cells[row, TargetFile.skuCell.Column].Value2 = "Не найден";
-                TargetFile.Sheet.Cells[row, TargetFile.oldSkuCell.Column].Value2 = positionAmount.Key.Sku;
+                TargetFile.Sheet.Cells[row, TargetFile.SkuCell.Column].Value2 = "Не найден";
+                TargetFile.Sheet.Cells[row, TargetFile.OldSkuCell.Column].Value2 = positionAmount.Key.Sku;
             }
 
             else
             {
-                TargetFile.Sheet.Cells[row, TargetFile.skuCell.Column].Value2 = positionAmount.Key.Sku;
+                TargetFile.Sheet.Cells[row, TargetFile.SkuCell.Column].Value2 = positionAmount.Key.Sku;
             }
 
             foreach (int column in columns)
@@ -146,7 +146,7 @@ namespace RehauSku.PriceListTools
 
             while (true)
             {
-                foundGroupValue = TargetFile.Sheet.Cells[found.Row, TargetFile.groupCell.Column].Value2.ToString();
+                foundGroupValue = TargetFile.Sheet.Cells[found.Row, TargetFile.GroupCell.Column].Value2.ToString();
 
                 if (string.IsNullOrEmpty(group) || group.Equals(foundGroupValue))
                 {
@@ -167,7 +167,7 @@ namespace RehauSku.PriceListTools
             AutoFilter filter = TargetFile.Sheet.AutoFilter;
             int startColumn = filter.Range.Column;
 
-            filter.Range.AutoFilter(TargetFile.amountCell.Column - startColumn + 1, "<>");
+            filter.Range.AutoFilter(TargetFile.AmountCell.Column - startColumn + 1, "<>");
             TargetFile.Sheet.Range["A1"].Activate();
         }
     }
diff --git a/src/PriceListTools/CombineTool.cs b/src/PriceListTools/CombineTool.cs
index e3cb83e..eddf9e7 100644
--- a/src/PriceListTools/CombineTool.cs
+++ b/src/PriceListTools/CombineTool.cs
@@ -33,17 +33,17 @@ namespace RehauSku.PriceListTools
 
             foreach (SourcePriceList source in SourceFiles)
             {
-                TargetFile.Sheet.Columns[TargetFile.amountCell.Column]
+                TargetFile.Sheet.Columns[TargetFile.AmountCell.Column]
                     .EntireColumn
                     .Insert(XlInsertShiftDirection.xlShiftToRight, XlInsertFormatOrigin.xlFormatFromRightOrBelow);
 
-                Range newColumnHeader = TargetFile.Sheet.Cells[TargetFile.amountCell.Row, TargetFile.amountCell.Column - 1];
+                Range newColumnHeader = TargetFile.Sheet.Cells[TargetFile.AmountCell.Row, TargetFile.AmountCell.Column - 1];
                 newColumnHeader.Value2 = $"{source.Name}";
                 newColumnHeader.WrapText = true;
 
                 foreach (var kvp in source.PositionAmount)
                 {
-                    FillPositionAmountToColumns(kvp, TargetFile.amountCell.Column - 1, TargetFile.amountCell.Column);
+                    FillPositionAmountToColumns(kvp, TargetFile.AmountCell.Column - 1, TargetFile.AmountCell.Column);
                     ProgressBar.Update();
                 }
             }
diff --git a/src/PriceListTools/ConvertTool.cs b/src/PriceListTools/ConvertTool.cs
index abf7e39..1bb02f4 100644
--- a/src/PriceListTools/ConvertTool.cs
+++ b/src/PriceListTools/ConvertTool.cs
@@ -18,7 +18,7 @@ namespace RehauSku.PriceListTools
 
             foreach (var kvp in Current.PositionAmount)
             {
-                FillPositionAmountToColumns(kvp, TargetFile.amountCell.Column);
+                FillPositionAmountToColumns(kvp, TargetFile.AmountCell.Column);
                 ProgressBar.Update();
             }
 
diff --git a/src/PriceListTools/ExportTool.cs b/src/PriceListTools/ExportTool.cs
index acc6515..603de8b 100644
--- a/src/PriceListTools/ExportTool.cs
+++ b/src/PriceListTools/ExportTool.cs
@@ -8,7 +8,7 @@ namespace RehauSku.PriceListTools
     internal class ExportTool : AbstractTool
     {
         private Dictionary<Position, double> PositionAmount;
-        private Range Selection;
+        private readonly Range Selection;
 
         public ExportTool()
         {
@@ -28,7 +28,7 @@ namespace RehauSku.PriceListTools
             
             foreach (var kvp in PositionAmount)
             {
-                FillPositionAmountToColumns(kvp, TargetFile.amountCell.Column);
+                FillPositionAmountToColumns(kvp, TargetFile.AmountCell.Column);
                 ProgressBar.Update();
             }
 
diff --git a/src/PriceListTools/MergeTool.cs b/src/PriceListTools/MergeTool.cs
index 77732bf..179fb81 100644
--- a/src/PriceListTools/MergeTool.cs
+++ b/src/PriceListTools/MergeTool.cs
@@ -33,7 +33,7 @@ namespace RehauSku.PriceListTools
             {
                 foreach (var kvp in source.PositionAmount)
                 {
-                    FillPositionAmountToColumns(kvp, TargetFile.amountCell.Column);
+                    FillPositionAmountToColumns(kvp, TargetFile.AmountCell.Column);
                     ProgressBar.Update();
                 }
             }
diff --git a/src/PriceListTools/PriceListHeaders.cs b/src/PriceListTools/PriceListHeaders.cs
new file mode 100644
index 0000000..74c7870
--- /dev/null
+++ b/src/PriceListTools/PriceListHeaders.cs
@@ -0,0 +1,11 @@
+namespace RehauSku.PriceListTools
+{
+    internal static class PriceListHeaders
+    {
+        public static readonly string Amount = "Кол-во";
+        public static readonly string OldSku = "Прежний материал";
+        public static readonly string Sku = "Актуальный материал";
+        public static readonly string Group = "Программа";
+        public static readonly string Name = "Наименование";
+    }
+}
\ No newline at end of file
diff --git a/src/PriceListTools/SourcePriceList.cs b/src/PriceListTools/SourcePriceList.cs
index 01637f8..d03d776 100644
--- a/src/PriceListTools/SourcePriceList.cs
+++ b/src/PriceListTools/SourcePriceList.cs
@@ -7,7 +7,6 @@ using RehauSku.Interface;
 
 namespace RehauSku.PriceListTools
 {
-
     internal class SourcePriceList : AbstractPriceList
     {
         public Dictionary<Position, double> PositionAmount { get; private set; }
@@ -24,10 +23,10 @@ namespace RehauSku.PriceListTools
 
             Range[] cells = new[]
             {
-                amountCell = Sheet.Cells.Find(amountHeader),
-                skuCell = Sheet.Cells.Find(skuHeader),
-                groupCell = Sheet.Cells.Find(groupHeader),
-                nameCell = Sheet.Cells.Find(nameHeader)
+                AmountCell = Sheet.Cells.Find(PriceListHeaders.Amount),
+                SkuCell = Sheet.Cells.Find(PriceListHeaders.Sku),
+                GroupCell = Sheet.Cells.Find(PriceListHeaders.Group),
+                NameCell = Sheet.Cells.Find(PriceListHeaders.Name)
             };
 
             if (cells.Any(x => x == null))
@@ -76,15 +75,15 @@ namespace RehauSku.PriceListTools
         {
             PositionAmount = new Dictionary<Position, double>();
 
-            for (int row = amountCell.Row + 1; row <= Sheet.Cells[Sheet.Rows.Count, amountCell.Column].End[XlDirection.xlUp].Row; row++)
+            for (int row = AmountCell.Row + 1; row <= Sheet.Cells[Sheet.Rows.Count, AmountCell.Column].End[XlDirection.xlUp].Row; row++)
             {
-                object amount = Sheet.Cells[row, amountCell.Column].Value2;
+                object amount = Sheet.Cells[row, AmountCell.Column].Value2;
 
                 if (amount != null && (double)amount != 0)
                 {
-                    object group = Sheet.Cells[row, groupCell.Column].Value2;
-                    object name = Sheet.Cells[row, nameCell.Column].Value2;
-                    object sku = Sheet.Cells[row, skuCell.Column].Value2;
+                    object group = Sheet.Cells[row, GroupCell.Column].Value2;
+                    object name = Sheet.Cells[row, NameCell.Column].Value2;
+                    object sku = Sheet.Cells[row, SkuCell.Column].Value2;
 
                     if (group == null || name == null || sku == null)
                         continue;
diff --git a/src/PriceListTools/TargetPriceList.cs b/src/PriceListTools/TargetPriceList.cs
index 5fb2bf9..2f23168 100644
--- a/src/PriceListTools/TargetPriceList.cs
+++ b/src/PriceListTools/TargetPriceList.cs
@@ -6,8 +6,7 @@ namespace RehauSku.PriceListTools
 {
     internal class TargetPriceList : AbstractPriceList
     {
-        private const string oldSkuHeader = "Прежний материал";
-        public Range oldSkuCell { get; private set; }
+        public Range OldSkuCell { get; private set; }
 
         public TargetPriceList(Workbook workbook)
         {
@@ -22,13 +21,13 @@ namespace RehauSku.PriceListTools
 
             Range[] cells = new[]
             {
-                amountCell = Sheet.Cells.Find(amountHeader),
-                skuCell = Sheet.Cells.Find(skuHeader),
-                groupCell = Sheet.Cells.Find(groupHeader),
-                nameCell = Sheet.Cells.Find(nameHeader)
+                AmountCell = Sheet.Cells.Find(PriceListHeaders.Amount),
+                SkuCell = Sheet.Cells.Find(PriceListHeaders.Sku),
+                GroupCell = Sheet.Cells.Find(PriceListHeaders.Group),
+                NameCell = Sheet.Cells.Find(PriceListHeaders.Name)
             };
 
-            oldSkuCell = Sheet.Cells.Find(oldSkuHeader);
+            OldSkuCell = Sheet.Cells.Find(PriceListHeaders.OldSku);
 
             if (cells.Any(x => x == null))
             {
diff --git a/src/RehauSku.Assist.csproj b/src/RehauSku.Assist.csproj
index 76bd243..8a2990b 100644
--- a/src/RehauSku.Assist.csproj
+++ b/src/RehauSku.Assist.csproj
@@ -131,6 +131,7 @@
     <Compile Include="PriceListTools\AbstractTool.cs" />
     <Compile Include="PriceListTools\MergeTool.cs" />
     <Compile Include="PriceListTools\AbstractPriceList.cs" />
+    <Compile Include="PriceListTools\PriceListHeaders.cs" />
     <Compile Include="PriceListTools\SourcePriceList.cs" />
     <Compile Include="PriceListTools\TargetPriceList.cs" />
     <Compile Include="Interface\RibbonController.cs" />

From bbfa09d863f7ca32faddb515cd796588ec759f49 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Sat, 12 Feb 2022 16:57:29 +0300
Subject: [PATCH 5/9] Button ids simplify

---
 src/Interface/RibbonController.cs | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/Interface/RibbonController.cs b/src/Interface/RibbonController.cs
index 7a58c7c..bd50550 100644
--- a/src/Interface/RibbonController.cs
+++ b/src/Interface/RibbonController.cs
@@ -20,11 +20,11 @@ namespace RehauSku.Interface
         <tabs>
           <tab id='rau' label='REHAU'>
             <group id='priceList' label='Прайс-лист'>
-                <button id='exportToPrice' getEnabled='GetExportEnabled' label='Экспорт в новый файл' size='normal' imageMso='PivotExportToExcel' onAction='OnToolPressed'/> 
-                <button id='convertPrice' getEnabled='GetConvertEnabled' label='Актуализировать' size='normal' imageMso='FileUpdate' onAction='OnToolPressed'/> 
+                <button id='export' getEnabled='GetExportEnabled' label='Экспорт в новый файл' size='normal' imageMso='PivotExportToExcel' onAction='OnToolPressed'/> 
+                <button id='convert' getEnabled='GetConvertEnabled' label='Актуализировать' size='normal' imageMso='FileUpdate' onAction='OnToolPressed'/> 
                 <menu id='conjoinMenu' label='Объединить' imageMso='Copy'>
-                    <button id='mergeFiles' label='Сложить' onAction='OnToolPressed'/>    
-                    <button id='combineFiles' label='По колонкам' onAction='OnToolPressed'/>   
+                    <button id='merge' label='Сложить' onAction='OnToolPressed'/>    
+                    <button id='combine' label='По колонкам' onAction='OnToolPressed'/>   
                 </menu>
             </group>
             <group id='rausettings' label='Настройки'>
@@ -65,16 +65,16 @@ namespace RehauSku.Interface
                 AbstractTool tool;
                 switch (control.Id)
                 {
-                    case "exportToPrice":
+                    case "export":
                         tool = new ExportTool();
                         break;
-                    case "convertPrice":
+                    case "convert":
                         tool = new ConvertTool();
                         break;
-                    case "mergeFiles":
+                    case "merge":
                         tool = new MergeTool();
                         break;
-                    case "combineFiles":
+                    case "combine":
                         tool = new CombineTool();
                         break;
                     default:

From 7144be9b9cae5fed0fcfde2e37512ab4397c6763 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Mon, 14 Feb 2022 10:34:26 +0300
Subject: [PATCH 6/9] Update EventsUtil to shortened button ids

---
 src/AddIn/EventsUtil.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/AddIn/EventsUtil.cs b/src/AddIn/EventsUtil.cs
index bc8d1d3..c10a69a 100644
--- a/src/AddIn/EventsUtil.cs
+++ b/src/AddIn/EventsUtil.cs
@@ -22,12 +22,12 @@ namespace RehauSku
 
         private static void RefreshConvertButton(object sh)
         {
-            Interface.RibbonController.RefreshControl("convertPrice");
+            Interface.RibbonController.RefreshControl("convert");
         }
 
         private static void RefreshExportButton(object sh, Range target)
         {
-            Interface.RibbonController.RefreshControl("exportToPrice");
+            Interface.RibbonController.RefreshControl("export");
         }
     }
 }

From 070733036b67e1b11132503d68a5e5d361cacc18 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Mon, 14 Feb 2022 11:06:40 +0300
Subject: [PATCH 7/9] getLabel implemetation for pricelist button

---
 src/AddIn/RegistryUtil.cs         |  6 ++++++
 src/Interface/RibbonController.cs | 18 ++++++++++++++++--
 2 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/src/AddIn/RegistryUtil.cs b/src/AddIn/RegistryUtil.cs
index 5fe2eea..a13e941 100644
--- a/src/AddIn/RegistryUtil.cs
+++ b/src/AddIn/RegistryUtil.cs
@@ -63,9 +63,15 @@ namespace RehauSku
             {
                 priceListPath = value;
                 RootKey.SetValue("PriceListPath", value);
+                RibbonController.RefreshControl("setPriceList");
             }
         }
 
+        public static string GetPriceListName()
+        {
+            return Path.GetFileName(priceListPath);
+        }
+
         public static ResponseOrder StoreResponseOrder
         {
             get
diff --git a/src/Interface/RibbonController.cs b/src/Interface/RibbonController.cs
index bd50550..7bf9ee1 100644
--- a/src/Interface/RibbonController.cs
+++ b/src/Interface/RibbonController.cs
@@ -2,6 +2,8 @@
 using Microsoft.Office.Interop.Excel;
 using RehauSku.PriceListTools;
 using System;
+using System.IO;
+using System.Reflection;
 using System.Runtime.InteropServices;
 using System.Windows.Forms;
 
@@ -27,8 +29,8 @@ namespace RehauSku.Interface
                     <button id='combine' label='По колонкам' onAction='OnToolPressed'/>   
                 </menu>
             </group>
-            <group id='rausettings' label='Настройки'>
-                <button id='setPriceList' label='Указать путь к шаблону' size='large' imageMso='CurrentViewSettings' onAction='OnSetPricePressed'/>
+            <group id='rausettings' getLabel='GetVersionLabel'>
+                <button id='setPriceList' getLabel='GetPriceListPathLabel' size='large' imageMso='TableExcelSpreadsheetInsert' onAction='OnSetPricePressed'/>
             </group>
           </tab>
         </tabs>
@@ -118,5 +120,17 @@ namespace RehauSku.Interface
                 return selection.Columns.Count == 2;
             }
         }
+
+        public string GetVersionLabel(IRibbonControl control)
+        {
+            string version = Assembly.GetExecutingAssembly().GetName().Version.ToString();
+            return $"v{version}";
+        }
+
+        public string GetPriceListPathLabel(IRibbonControl control)
+        {
+            string name = RegistryUtil.GetPriceListName();
+            return string.IsNullOrEmpty(name) ? "Нет файла шаблона!" : name;
+        }
     }
 }

From 3d30e6a77bb419b8afb4cc804f59fdbd6751d97d Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Mon, 14 Feb 2022 11:06:53 +0300
Subject: [PATCH 8/9] Change Assembly version

---
 src/Properties/AssemblyInfo.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Properties/AssemblyInfo.cs b/src/Properties/AssemblyInfo.cs
index b0cc560..db45d3f 100644
--- a/src/Properties/AssemblyInfo.cs
+++ b/src/Properties/AssemblyInfo.cs
@@ -32,5 +32,5 @@ using System.Runtime.InteropServices;
 // You can specify all the values or you can default the Build and Revision Numbers
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("1.0.0.0")]
-[assembly: AssemblyFileVersion("1.0.0.0")]
+[assembly: AssemblyVersion("1.0.4")]
+[assembly: AssemblyFileVersion("1.0.4")]

From f190e27948255303c73a6b457ad1c3af1c88dba9 Mon Sep 17 00:00:00 2001
From: Sergey Chebotar <s.chebotar@gmail.com>
Date: Mon, 14 Feb 2022 11:07:38 +0300
Subject: [PATCH 9/9] Change copyright dates

---
 src/Properties/AssemblyInfo.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Properties/AssemblyInfo.cs b/src/Properties/AssemblyInfo.cs
index db45d3f..b348361 100644
--- a/src/Properties/AssemblyInfo.cs
+++ b/src/Properties/AssemblyInfo.cs
@@ -10,7 +10,7 @@ using System.Runtime.InteropServices;
 [assembly: AssemblyConfiguration("")]
 [assembly: AssemblyCompany("")]
 [assembly: AssemblyProduct("RehauSku.Assist")]
-[assembly: AssemblyCopyright("Copyright ©  2021")]
+[assembly: AssemblyCopyright("Copyright ©  2021-2022")]
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]