* Bug 16013 fixed: load environment does not fail any more 04/20904/5
St├ęphane Mottelet [Fri, 22 Mar 2019 14:20:42 +0000 (15:20 +0100)]
http://bugzilla.scilab.org/show_bug.cgi?id=16013

Change-Id: I6fae79b58a7ffc5203b1f0e10b2f290142ee7d76

scilab/CHANGES.md
scilab/modules/gui/etc/main_menubar.xml
scilab/modules/gui/tests/nonreg_tests/bug_16013.tst [new file with mode: 0644]

index e2ca3b2..53e6532 100644 (file)
@@ -159,6 +159,7 @@ Bug Fixes
 * [#16005](http://bugzilla.scilab.org/show_bug.cgi?id=16005): The `intdec` example was biased and not robust when changing sampling frequencies.
 * [#16007](http://bugzilla.scilab.org/show_bug.cgi?id=16007): Non-integer index in sparse makes Scilab crash.
 * [#16012](http://bugzilla.scilab.org/show_bug.cgi?id=16012): `[struct() struct()]` crashed Scilab.
+* [#16013](http://bugzilla.scilab.org/show_bug.cgi?id=16013): Load previously saved environment with "File/Load environment" menu failed.
 * [#16014](http://bugzilla.scilab.org/show_bug.cgi?id=16014): after `x.a=1; x(:)=[]` x.a was an empty list. 
 * [#16021](http://bugzilla.scilab.org/show_bug.cgi?id=16021): `tand([-90 90])` answered [Nan Nan] instead of [-Inf, Inf]. `cotd([-90 90])` answered [Nan Nan] instead of [0 0]. `1 ./cosd([-90 90])` answered [Inf -Inf] instead of [Inf Inf].
 * [#16071](http://bugzilla.scilab.org/show_bug.cgi?id=16071): `prettyprint(complex(1,%nan))` omitted the "+" in `1 + Nani`. It printed positive exponents with a useless "+". For any input nul polynomial, the string result never included the name of the variable. Default input arguments could not be skipped. ExportFormat was uselessly case-sensitive. For tex|latex: for text input, $ \ % & { } ~ and ^ special characters were not protected ; spaces were not protected, all words were concatenated ; for polynomials and rationals, the result string could be extremely long and not easily wrappable. For MathML: "<" was not protected ; <mi></mi> were missing for text input ; <mtable>, </mtable>, <mtr>, </mtr>, <mtd>, <mfenced> and </mfenced> tags were not wrapped and could not be indented. Delimiters: "" was not documented as possible value ; ")" was wrongly documented. Dynamical linear systems were not documented as possible input.
index f5dceb4..3f64dc7 100644 (file)
@@ -16,7 +16,7 @@
             <callback instruction="%fileToLoad = uigetfile([&quot;*.sav&quot;;&quot;*.sod&quot;], &quot;&quot;, gettext(&quot;Select a file to load&quot;)); if ~(%fileToLoad == &quot;&quot;) then load(%fileToLoad); updatebrowsevar(); disp(msprintf(gettext(&quot;Environment loaded.\n&quot;))); end; clear(&quot;%fileToLoad&quot;);" type="0"/>
         </submenu>
         <submenu label="&amp;Save environment..." accelerator="OSSCKEY S" icon="document-save">
-            <callback instruction="%fileToSave = uiputfile([&quot;*.sav&quot;;&quot;*.sod&quot;], &quot;&quot;, gettext(&quot;Select a file to write&quot;)); if ~(%fileToSave == &quot;&quot;) then if (fileparts(%fileToSave, &quot;extension&quot;) == &quot;&quot;) then %fileToSave = %fileToSave + &quot;.sav&quot;; end; save(%fileToSave); disp(msprintf(gettext(&quot;Environment saved.\n&quot;))); end; clear(&quot;%fileToSave&quot;);" type="0"/>
+            <callback instruction="%fileToSave = uiputfile([&quot;*.sav&quot;;&quot;*.sod&quot;], &quot;&quot;, gettext(&quot;Select a file to write&quot;)); if ~(%fileToSave == &quot;&quot;) then if (fileparts(%fileToSave, &quot;extension&quot;) == &quot;&quot;) then %fileToSave = %fileToSave + &quot;.sav&quot;; end; clear gcbo; execstr(msprintf(&quot;clear %%fileToSave\n save %s&quot;,%fileToSave)); disp(msprintf(gettext(&quot;Environment saved.\n&quot;))); end;" type="0"/>
         </submenu>
         <separator/>
         <submenu label="&amp;Current Working Directory" icon="scilab-folder-open">
diff --git a/scilab/modules/gui/tests/nonreg_tests/bug_16013.tst b/scilab/modules/gui/tests/nonreg_tests/bug_16013.tst
new file mode 100644 (file)
index 0000000..48ee7cf
--- /dev/null
@@ -0,0 +1,23 @@
+// =============================================================================
+// Scilab ( http://www.scilab.org/ ) - This file is part of Scilab
+// Copyright (C) 2019 - St├ęphane MOTTELET
+//
+//  This file is distributed under the same license as the Scilab package.
+// =============================================================================
+//
+// <-- TEST WITH GRAPHICS -->
+// <-- INTERACTIVE TEST -->
+// <-- NO CHECK REF -->
+//
+// <-- Non-regression test for bug 16013 -->
+//
+// <-- Bugzilla URL -->
+// http://bugzilla.scilab.org/show_bug.cgi?id=16013
+//
+// <-- Short Description -->
+// load environment fails
+
+// Select File/Save environment and choose a file name, e.g. session.sav
+// then select File/Load environment, select session.sav and check that no error is raised
+
+