From 13a79663e6fcba2c659817fcea2873d29ec0cedf Mon Sep 17 00:00:00 2001
From: JuliaSvinareva <Julia.Svinaryova@onlyoffice.com>
Date: Wed, 6 Apr 2022 22:32:50 +0300
Subject: [PATCH] [SSE] Make select range setting in search panel

---
 apps/common/main/lib/view/SearchPanel.js      |  9 +--
 .../main/app/controller/Search.js             | 58 +++++++++++++++++--
 2 files changed, 59 insertions(+), 8 deletions(-)

diff --git a/apps/common/main/lib/view/SearchPanel.js b/apps/common/main/lib/view/SearchPanel.js
index 61c6ed5814..48d96bfb9c 100644
--- a/apps/common/main/lib/view/SearchPanel.js
+++ b/apps/common/main/lib/view/SearchPanel.js
@@ -189,15 +189,16 @@ define([
                         me.fireEvent('search:options', ['within', record.value]);
                     });
 
-                    this.inputSelectRange = new Common.UI.InputField({
+                    this.inputSelectRange = new Common.UI.InputFieldBtn({
                         el: $('#search-adv-select-range'),
                         placeHolder: this.textSelectDataRange,
                         allowBlank: true,
                         validateOnChange: true,
+                        validateOnBlur: true,
                         style: "width: 219px; margin-top: 8px",
                         disabled: true
-                    }).on('changed:after', function(input, newValue) {
-                        me.fireEvent('search:options', ['range', newValue]);
+                    }).on('keyup:after', function(input, e) {
+                        me.fireEvent('search:options', ['range', input.getValue(), e.keyCode !== Common.UI.Keys.RETURN]);
                     });
 
                     this.cmbSearch = new Common.UI.ComboBox({
@@ -266,7 +267,7 @@ define([
 
         focus: function(type) {
             var me  = this,
-                el = type === 'replace' ? me.inputReplace.$el : me.inputText.$el;
+                el = type === 'replace' ? me.inputReplace.$el : (type === 'range' ? me.inputSelectRange.$el : me.inputText.$el);
             setTimeout(function(){
                 el.find('input').focus();
                 el.find('input').select();
diff --git a/apps/spreadsheeteditor/main/app/controller/Search.js b/apps/spreadsheeteditor/main/app/controller/Search.js
index 2ef5bb1207..2c63a74795 100644
--- a/apps/spreadsheeteditor/main/app/controller/Search.js
+++ b/apps/spreadsheeteditor/main/app/controller/Search.js
@@ -118,13 +118,15 @@ define([
                 if (_.isEmpty(value)) {
                     return true;
                 }
-                var isvalid = me.api.asc_checkDataRange(Asc.c_oAscSearchBy.Range, value);
-                me._state.isValidSelectedRange = isvalid;
+                var isvalid = me.api.asc_checkDataRange(undefined, value);
+                me._state.isValidSelectedRange = isvalid !== Asc.c_oAscError.ID.DataRangeError;
                 return (isvalid === Asc.c_oAscError.ID.DataRangeError) ? me.textInvalidRange : true;
             };
+            this.view.inputSelectRange.on('button:click', _.bind(this.onRangeSelect, this));
         },
 
-        onChangeSearchOption: function (option, value) {
+        onChangeSearchOption: function (option, value, noSearch) {
+            var runSearch = true;
             switch (option) {
                 case 'case-sensitive':
                     this._state.matchCase = value;
@@ -138,9 +140,13 @@ define([
                 case 'within':
                     this._state.withinSheet = value === 0 ? Asc.c_oAscSearchBy.Sheet : (value === 1 ? Asc.c_oAscSearchBy.Workbook : Asc.c_oAscSearchBy.Range);
                     this.view.inputSelectRange.setDisabled(value !== Asc.c_oAscSearchBy.Range);
+                    if (value === Asc.c_oAscSearchBy.Range) {
+                        runSearch = false;
+                    }
                     break;
                 case 'range':
                     this._state.selectedRange = value;
+                    runSearch = !noSearch;
                     break;
                 case 'search':
                     this._state.searchByRows = value;
@@ -148,8 +154,39 @@ define([
                 case 'lookIn':
                     this._state.lookInFormulas = value;
                     break;
-
             }
+            if (runSearch && this._state.searchText !== '' && this.onQuerySearch()) {
+                this.hideResults();
+                clearInterval(this.searchTimer);
+                this.searchTimer = undefined;
+                this.api.asc_StartTextAroundSearch();
+            }
+        },
+
+        onRangeSelect: function () {
+            var me = this;
+            var handlerDlg = function(dlg, result) {
+                if (result == 'ok') {
+                    var valid = dlg.getSettings();
+                    me.view.inputSelectRange.setValue(valid);
+                    me.view.inputSelectRange.checkValidate();
+                    me.onChangeSearchOption('range', valid);
+                }
+            };
+
+            var win = new SSE.Views.CellRangeDialog({
+                handler: handlerDlg
+            }).on('close', function() {
+                _.delay(function(){
+                    me.view.inputSelectRange.focus();
+                },1);
+            });
+            win.show();
+            win.setSettings({
+                api: me.api,
+                range: (!_.isEmpty(me.view.inputSelectRange.getValue()) && (me.view.inputSelectRange.checkValidate()==true)) ? me.view.inputSelectRange.getValue() : me._state.selectedRange,
+                type: Asc.c_oAscSelectionDialogType.PrintTitles
+            });
         },
 
         onSearchNext: function (type, text, e) {
@@ -194,6 +231,19 @@ define([
         },
 
         onQuerySearch: function (d, w, opts, fromPanel) {
+            var me = this;
+            if (this._state.withinSheet === Asc.c_oAscSearchBy.Range && !this._state.isValidSelectedRange) {
+                Common.UI.warning({
+                    title: this.notcriticalErrorTitle,
+                    msg: this.textInvalidRange,
+                    buttons: ['ok'],
+                    callback: function () {
+                        me.view.focus('range');
+                    }
+                });
+                return;
+            }
+
             var options = new Asc.asc_CFindOptions();
             options.asc_setFindWhat(this._state.searchText);
             options.asc_setScanForward(d != 'back');