[IMP] stock_request + stock_request_submit: Change the approach to Confirm and Submit buttons

Related to OCA#1514 (comment)

Author: victoralmau

stock_request_submit/views/stock_request_order_views.xml

@@ -7,19 +7,21 @@
         <field name="model">stock.request.order</field>
         <field name="inherit_id" ref="stock_request.stock_request_order_form" />
         <field name="arch" type="xml">
-            <button name="action_confirm" position="after">
+            <button name="action_confirm" position="attributes">
+                <attribute name="name">action_submit</attribute>
+                <attribute
+                    name="attrs"
+                >{'invisible': [('state', '!=', 'draft')]}</attribute>
+            </button>
+            <button name="action_submit" position="after">
                 <button
-                    name="action_submit"
+                    name="action_confirm"
                     string="Submit"
                     type="object"
-                    attrs="{'invisible': [('state', '!=', 'draft')]}"
+                    attrs="{'invisible': [('state', '!=', 'submitted')]}"
+                    groups="stock_request.group_stock_request_manager"
                 />
             </button>
-            <button name="action_confirm" position="attributes">
-                <attribute
-                    name="attrs"
-                >{'invisible': [('state', '!=', 'submitted')]}</attribute>
-            </button>
             <button name="action_draft" position="attributes">
                 <attribute name="states">submitted,cancel</attribute>
             </button>