README.md

@@ -18,7 +18,6 @@ modelling of rivers.
 + Define river sedimentary layers
 + Save the study in a unique file
 + Run Mage 8 on a study and visualize results
-+ Create and edit study alternative scenario tree
 
 Let see the
 [documentation](https://gitlab.irstea.fr/theophile.terraz/pamhyr/-/wikis/home)

doc/dev/build.sh

@@ -1,23 +1,18 @@
 #! /bin/sh
 
-FILES="documentation.org scenario.org"
-
-for file in $FILES
-do
-    echo " PDF  $file"
+echo " PDF  documentation.org"
 emacs -nw --batch                                       \
       --eval "(require 'org)"                           \
       --eval "(require 'ox-latex)"                      \
       --eval "(load-file \"../tools/setup.el\")"        \
-          --file ./$file                                \
-          -f org-latex-export-to-pdf                    \
+      --file ./documentation.org                        \
+      -f org-latex-export-to-pdf
 
-    echo " HTML $file"
+echo " HTML documentation.org"
 emacs -nw --batch                                       \
       --eval "(require 'org)"                           \
       --eval "(require 'ox-latex)"                      \
       --eval "(require 'ox-html)"                       \
       --eval "(load-file \"../tools/setup.el\")"        \
-          --file ./$file                                \
+      --file ./documentation.org                        \
       -f org-html-export-to-html
-done

src/Model/Reservoir/Reservoir.py

@@ -157,7 +157,7 @@ class Data(SQLSubModel):
                 owner_scenario=owner_scenario
             )
             if deleted:
-                nd.set_as_deleted()
+                f.set_as_deleted()
 
             nd._data = [elevation, surface]
 
@@ -307,7 +307,7 @@ class Reservoir(SQLSubModel):
                 owner_scenario=owner_scenario
             )
             if deleted:
-                new_reservoir.set_as_deleted()
+                f.set_as_deleted()
 
             new_reservoir._node = None
             if node_id != -1:

src/View/Network/GraphWidget.py

@@ -73,13 +73,10 @@ class NodeItem(QGraphicsItem):
 
     def shape(self):
         path = QPainterPath()
-        reservoir = self.graph.graph._reservoir.get_assoc_to_node(self.node)
-
-        if reservoir is None or reservoir.is_deleted():
+        if self.graph.graph._reservoir.get_assoc_to_node(self.node) is None:
             path.addEllipse(-10, -10, 20, 20)
         else:
             path.addRect(-10, -10, 20, 20)
-
         return path
 
     @timer
@@ -101,9 +98,7 @@ class NodeItem(QGraphicsItem):
             color = Qt.green
 
         painter.setBrush(QBrush(color))
-        reservoir = self.graph.graph._reservoir.get_assoc_to_node(self.node)
-
-        if reservoir is None or reservoir.is_deleted():
+        if self.graph.graph._reservoir.get_assoc_to_node(self.node) is None:
             painter.drawEllipse(-10, -10, 20, 20)
         else:
             painter.drawRect(-10, -10, 20, 20)

src/View/Reservoir/Table.py

@@ -38,7 +38,7 @@ from View.Tools.PamhyrTable import PamhyrTableModel
 
 from View.Reservoir.UndoCommand import (
     SetNameCommand, SetNodeCommand,
-    AddCommand, DelCommand,
+    AddCommand, DelCommand, PasteCommand,
 )
 
 logger = logging.getLogger()
@@ -103,19 +103,6 @@ class TableModel(PamhyrTableModel):
     def _setup_lst(self):
         self._lst = self._data.reservoir
 
-    def get_true_data_row(self, row):
-        resvervoir = self._lst.get(row)
-
-        return next(
-            map(
-                lambda e: e[0],
-                filter(
-                    lambda e: e[1] == resvervoir,
-                    enumerate(self._lst.lst)
-                )
-            ), 0
-        )
-
     def rowCount(self, parent):
         return len(self._lst)
 
@@ -166,8 +153,6 @@ class TableModel(PamhyrTableModel):
     def add(self, row, parent=QModelIndex()):
         self.beginInsertRows(parent, row, row - 1)
 
-        row = self.get_true_data_row(row)
-
         self._undo.push(
             AddCommand(
                 self._lst, row
@@ -182,12 +167,7 @@ class TableModel(PamhyrTableModel):
 
         self._undo.push(
             DelCommand(
-                self._lst, list(
-                    map(
-                        lambda r: self._lst.get(r),
-                        rows
-                    )
-                )
+                self._lst, rows
             )
         )
 

src/View/Reservoir/UndoCommand.py

@@ -76,8 +76,7 @@ class AddCommand(QUndoCommand):
         self._new = None
 
     def undo(self):
-        self._new.set_as_deleted()
-        self._reservoir_lst.modified()
+        self._reservoir_lst.delete_i([self._index])
 
     def redo(self):
         if self._new is None:
@@ -85,8 +84,6 @@ class AddCommand(QUndoCommand):
         else:
             self._reservoir_lst.insert(self._index, self._new)
 
-        self._reservoir_lst.modified()
-
 
 class AddAndAssociateCommand(QUndoCommand):
     def __init__(self, reservoir_lst, index, node):
@@ -99,8 +96,7 @@ class AddAndAssociateCommand(QUndoCommand):
         self._node = node
 
     def undo(self):
-        self._reservoir_lst.delete(self._new)
-        self._reservoir_lst.modified()
+        self._reservoir_lst.delete_i([self._index])
 
     def redo(self):
         if self._new is None:
@@ -109,44 +105,45 @@ class AddAndAssociateCommand(QUndoCommand):
             self._reservoir_lst.insert(self._index, self._new)
 
         self._reservoir_lst.get(self._index).node = self._node
-        self._reservoir_lst.modified()
 
 
 class DelCommand(QUndoCommand):
-    def __init__(self, reservoir_lst, reservoirs):
+    def __init__(self, reservoir_lst, rows):
         QUndoCommand.__init__(self)
 
         self._reservoir_lst = reservoir_lst
-        self._reservoirs = reservoirs
+
+        self._rows = rows
+
+        self._reservoir = []
+        for row in rows:
+            self._reservoir.append((row, self._reservoir_lst.get(row)))
+        self._reservoir.sort()
 
     def undo(self):
-        for reservoir in self._reservoirs:
-            reservoir.set_as_not_deleted()
-
-        self._reservoir_lst.modified()
+        for row, el in self._reservoir:
+            self._reservoir_lst.insert(row, el)
 
     def redo(self):
-        for reservoir in self._reservoirs:
-            reservoir.set_as_deleted()
+        self._reservoir_lst.delete_i(self._rows)
 
-        self._reservoir_lst.modified()
 
-# class PasteCommand(QUndoCommand):
-#     def __init__(self, reservoir_lst, row, reservoir):
-#         QUndoCommand.__init__(self)
+class PasteCommand(QUndoCommand):
+    def __init__(self, reservoir_lst, row, reservoir):
+        QUndoCommand.__init__(self)
 
-#         self._reservoir_lst = reservoir_lst
+        self._reservoir_lst = reservoir_lst
 
-#         self._row = row
-#         self._reservoir = deepcopy(reservoir)
-#         self._reservoir.reverse()
+        self._row = row
+        self._reservoir = deepcopy(reservoir)
+        self._reservoir.reverse()
 
-#     def undo(self):
-#         self._reservoir_lst.delete_i(
-#             self._tab,
-#             range(self._row, self._row + len(self._reservoir))
-#         )
+    def undo(self):
+        self._reservoir_lst.delete_i(
+            self._tab,
+            range(self._row, self._row + len(self._reservoir))
+        )
 
-#     def redo(self):
-#         for r in self._reservoir:
-#             self._reservoir_lst.insert(self._row, r)
+    def redo(self):
+        for r in self._reservoir:
+            self._reservoir_lst.insert(self._row, r)