From b1568a7319e7fd3b6fb49b41b9a6be6d261ae14b Mon Sep 17 00:00:00 2001
From: Ion Agorria <ion@agorria.com>
Date: Sun, 14 Feb 2016 14:46:58 +0100
Subject: [PATCH] Better Add buttons in dynamicprice module

---
 htdocs/langs/en_US/products.lang        |  2 ++
 htdocs/product/admin/dynamic_prices.php | 23 ++++++++++++++++++-----
 2 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/htdocs/langs/en_US/products.lang b/htdocs/langs/en_US/products.lang
index 3dfabdc124d..bf223e0e33e 100644
--- a/htdocs/langs/en_US/products.lang
+++ b/htdocs/langs/en_US/products.lang
@@ -296,6 +296,8 @@ ComposedProduct=Sub-product
 MinSupplierPrice=Minimum supplier price
 DynamicPriceConfiguration=Dynamic price configuration
 DynamicPriceDesc=On product card, with this module enabled, you should be able to set mathematic functions to calculate Customer or Supplier prices. Such function can use all mathematic operators, some constants and variables. You can set here the variables you want to be able and if the variable need an automatic update, the external URL to use to ask Dolibarr to update automaticaly the value. 
+AddVariable=Add Variable
+AddUpdater=Add Updater
 GlobalVariables=Global variables
 VariableToUpdate=Variable to update
 GlobalVariableUpdaters=Global variable updaters
diff --git a/htdocs/product/admin/dynamic_prices.php b/htdocs/product/admin/dynamic_prices.php
index 84ec54433cd..7c9a46d8972 100644
--- a/htdocs/product/admin/dynamic_prices.php
+++ b/htdocs/product/admin/dynamic_prices.php
@@ -173,6 +173,16 @@ if ($action != 'create_updater' && $action != 'edit_updater') {
         print '</tr>';
     }
     print '</table>';
+
+    if (empty($action))
+    {
+        //Action Buttons
+        print '<div class="tabsAction">';
+        print '<a class="butAction" href="'.$_SERVER["PHP_SELF"].'?action=create_variable">'.$langs->trans("AddVariable").'</a>';
+        print '</div>';
+        //Separator is only need for updaters table is showed after buttons
+        print '<br><br>';
+    }
 }
 
 //Global variable editor
@@ -253,6 +263,14 @@ if ($action != 'create_variable' && $action != 'edit_variable') {
         print '</tr>';
     }
     print '</table>';
+
+    if (empty($action))
+    {
+        //Action Buttons
+        print '<div class="tabsAction">';
+        print '<a class="butAction" href="'.$_SERVER["PHP_SELF"].'?action=create_updater">'.$langs->trans("AddUpdater").'</a>';
+        print '</div>';
+    }
 }
 
 //Updater editor
@@ -321,11 +339,6 @@ if ($action == 'create_updater' || $action == 'edit_updater') {
     print '<input type="submit" class="button" name="cancel" id="cancel" value="'.$langs->trans("Cancel").'">';
     print '</div>';
     print '</form>';
-} else {
-    //Action Buttons
-    print '<div class="tabsAction">';
-    print '<a class="butAction" href="'.$_SERVER["PHP_SELF"].'?action=create_updater">'.$langs->trans("Add").'</a>';
-    print '</div>';
 }
 
 llxFooter();