A fix to make editinplace working better

This commit is contained in:
Laurent Destailleur
2016-07-28 15:29:46 +02:00
parent 4e3dd4c927
commit 00118610bb
4 changed files with 30 additions and 12 deletions

View File

@@ -1836,9 +1836,9 @@ if ($action == 'create')
// Delivery date // Delivery date
$langs->load('deliveries'); $langs->load('deliveries');
print '<tr><td>'; print '<tr><td>';
print $form->editfieldkey($langs->trans('DeliveryDate'), 'date_livraison', $object->date_livraison, $object, $user->rights->propal->creer); print $form->editfieldkey($langs->trans('DeliveryDate'), 'date_livraison', $object->date_livraison, $object, $user->rights->propal->creer, 'datepicker');
print '</td><td colspan="5">'; print '</td><td colspan="5">';
print $form->editfieldval($langs->trans('DeliveryDate'), 'date_livraison', $object->date_livraison, $object, $user->rights->propal->creer, 'day'); print $form->editfieldval($langs->trans('DeliveryDate'), 'date_livraison', $object->date_livraison, $object, $user->rights->propal->creer, 'datepicker');
print '</td>'; print '</td>';
print '</tr>'; print '</tr>';

View File

@@ -77,7 +77,7 @@ class Form
* @param string $preselected Value to show/edit (not used in this function) * @param string $preselected Value to show/edit (not used in this function)
* @param object $object Object * @param object $object Object
* @param boolean $perm Permission to allow button to edit parameter. Set it to 0 to have a not edited field. * @param boolean $perm Permission to allow button to edit parameter. Set it to 0 to have a not edited field.
* @param string $typeofdata Type of data ('string' by default, 'email', 'amount:99', 'numeric:99', 'text' or 'textarea:rows:cols', 'day' or 'datepicker', 'ckeditor:dolibarr_zzz:width:height:savemethod:1:rows:cols', 'select;xxx[:class]'...) * @param string $typeofdata Type of data ('string' by default, 'email', 'amount:99', 'numeric:99', 'text' or 'textarea:rows:cols', 'datepicker' ('day' do not work, don't know why), 'ckeditor:dolibarr_zzz:width:height:savemethod:1:rows:cols', 'select;xxx[:class]'...)
* @param string $moreparam More param to add on a href URL* * @param string $moreparam More param to add on a href URL*
* @param int $fieldrequired 1 if we want to show field as mandatory using the fieldrequired CSS. * @param int $fieldrequired 1 if we want to show field as mandatory using the fieldrequired CSS.
* @return string HTML edit field * @return string HTML edit field
@@ -129,7 +129,7 @@ class Form
* @param string $value Value to show/edit * @param string $value Value to show/edit
* @param object $object Object * @param object $object Object
* @param boolean $perm Permission to allow button to edit parameter * @param boolean $perm Permission to allow button to edit parameter
* @param string $typeofdata Type of data ('string' by default, 'email', 'amount:99', 'numeric:99', 'text' or 'textarea:rows:cols', 'day' or 'datepicker', 'dayhour' or 'datepickerhour', 'ckeditor:dolibarr_zzz:width:height:savemethod:toolbarstartexpanded:rows:cols', 'select:xxx'...) * @param string $typeofdata Type of data ('string' by default, 'email', 'amount:99', 'numeric:99', 'text' or 'textarea:rows:cols', 'datepicker' ('day' do not work, don't know why), 'dayhour' or 'datepickerhour', 'ckeditor:dolibarr_zzz:width:height:savemethod:toolbarstartexpanded:rows:cols', 'select:xxx'...)
* @param string $editvalue When in edit mode, use this value as $value instead of value (for example, you can provide here a formated price instead of value). Use '' to use same than $value * @param string $editvalue When in edit mode, use this value as $value instead of value (for example, you can provide here a formated price instead of value). Use '' to use same than $value
* @param object $extObject External object * @param object $extObject External object
* @param mixed $custommsg String or Array of custom messages : eg array('success' => 'MyMessage', 'error' => 'MyMessage') * @param mixed $custommsg String or Array of custom messages : eg array('success' => 'MyMessage', 'error' => 'MyMessage')
@@ -254,7 +254,7 @@ class Form
* @param string $value Value to show/edit * @param string $value Value to show/edit
* @param string $htmlname DIV ID (field name) * @param string $htmlname DIV ID (field name)
* @param int $condition Condition to edit * @param int $condition Condition to edit
* @param string $inputType Type of input ('string', 'numeric', 'datepicker', 'textarea:rows:cols', 'ckeditor:dolibarr_zzz:width:height:?:1:rows:cols', 'select:xxx') * @param string $inputType Type of input ('string', 'numeric', 'datepicker' ('day' do not work, don't know why), 'textarea:rows:cols', 'ckeditor:dolibarr_zzz:width:height:?:1:rows:cols', 'select:xxx')
* @param string $editvalue When in edit mode, use this value as $value instead of value * @param string $editvalue When in edit mode, use this value as $value instead of value
* @param object $extObject External object * @param object $extObject External object
* @param mixed $custommsg String or Array of custom messages : eg array('success' => 'MyMessage', 'error' => 'MyMessage') * @param mixed $custommsg String or Array of custom messages : eg array('success' => 'MyMessage', 'error' => 'MyMessage')
@@ -269,7 +269,7 @@ class Form
// Check parameters // Check parameters
if ($inputType == 'textarea') $value = dol_nl2br($value); if ($inputType == 'textarea') $value = dol_nl2br($value);
else if (preg_match('/^numeric/',$inputType)) $value = price($value); else if (preg_match('/^numeric/',$inputType)) $value = price($value);
else if ($inputType == 'datepicker') $value = dol_print_date($value, 'day'); else if ($inputType == 'day' || $inputType == 'datepicker') $value = dol_print_date($value, 'day');
if ($condition) if ($condition)
{ {
@@ -302,7 +302,7 @@ class Form
if (! empty($tmp[2])) $savemethod=$tmp[2]; if (! empty($tmp[2])) $savemethod=$tmp[2];
$out.= '<input id="width_'.$htmlname.'" value="'.$inputOption.'" type="hidden"/>'."\n"; $out.= '<input id="width_'.$htmlname.'" value="'.$inputOption.'" type="hidden"/>'."\n";
} }
else if ((preg_match('/^datepicker/',$inputType)) || (preg_match('/^datehourpicker/',$inputType))) else if ((preg_match('/^day$/',$inputType)) || (preg_match('/^datepicker/',$inputType)) || (preg_match('/^datehourpicker/',$inputType)))
{ {
$tmp=explode(':',$inputType); $tmp=explode(':',$inputType);
$inputType=$tmp[0]; $inputType=$tmp[0];

View File

@@ -53,9 +53,11 @@ $(document).ready(function() {
}); });
$('.editkey_textarea').hover( $('.editkey_textarea').hover(
function () { function () {
console.log("We are hover (entry) an editkey_textarea");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover (exit) an editkey_textarea");
$('#viewval_' + $(this).attr('id')).removeClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).removeClass("viewval_hover");
} }
); );
@@ -104,9 +106,11 @@ $(document).ready(function() {
}); });
$('.editkey_ckeditor').hover( $('.editkey_ckeditor').hover(
function () { function () {
console.log("We are hover (entry) an editkey_ckeditor");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover (exit) an editkey_ckeditor");
$('#viewval_' + $(this).attr('id')).removeClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).removeClass("viewval_hover");
} }
); );
@@ -145,9 +149,11 @@ $(document).ready(function() {
}); });
$('.editkey_string').hover( $('.editkey_string').hover(
function () { function () {
console.log("We are hover (entry) an editkey_string");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover an editkey_string");
$('#viewval_' + $(this).attr('id')).removeClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).removeClass("viewval_hover");
} }
); );
@@ -186,9 +192,11 @@ $(document).ready(function() {
}); });
$('.editkey_numeric').hover( $('.editkey_numeric').hover(
function () { function () {
console.log("We are hover an editkey_numeric");
$( '#viewval_' + $(this).attr('id') ).addClass("viewval_hover"); $( '#viewval_' + $(this).attr('id') ).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover (exit) an editkey_textarea");
$( '#viewval_' + $(this).attr('id') ).removeClass("viewval_hover"); $( '#viewval_' + $(this).attr('id') ).removeClass("viewval_hover");
} }
); );
@@ -227,9 +235,11 @@ $(document).ready(function() {
}); });
$('.editkey_datepicker').hover( $('.editkey_datepicker').hover(
function () { function () {
console.log("We are hover (entry) editkey_datepicker");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover (exit) an editkey_datepicker");
$('#viewval_' + $(this).attr('id')).removeClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).removeClass("viewval_hover");
} }
); );
@@ -270,9 +280,11 @@ $(document).ready(function() {
}); });
$('.editkey_select').hover( $('.editkey_select').hover(
function () { function () {
console.log("We are hover (entry) an editkey_select");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {
console.log("We are no more hover (exit) an editkey_select");
$('#viewval_' + $(this).attr('id')).removeClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).removeClass("viewval_hover");
} }
); );
@@ -316,6 +328,7 @@ $(document).ready(function() {
}); });
$('.editkey_autocomplete').hover( $('.editkey_autocomplete').hover(
function () { function () {
console.log("We are no more hover (exit) an editkey_autocomplete");
$('#viewval_' + $(this).attr('id')).addClass("viewval_hover"); $('#viewval_' + $(this).attr('id')).addClass("viewval_hover");
}, },
function () { function () {

View File

@@ -51,8 +51,11 @@ if (in_array($object->element,array('propal', 'supplier_proposal','facture','fac
?> ?>
<!-- BEGIN PHP TEMPLATE objectline_create.tpl.php --> <!-- BEGIN PHP TEMPLATE objectline_create.tpl.php -->
<?php $nolinesbefore=(count($this->lines) == 0); ?> <?php
<tr class="liste_titre<?php echo ($nolinesbefore?'':' liste_titre_add') ?> nodrag nodrop"> $nolinesbefore=(count($this->lines) == 0);
if ($nolinesbefore) {
?>
<tr class="liste_titre nodrag nodrop">
<td class="linecoldescription" <?php echo (! empty($conf->global->MAIN_VIEW_LINE_NUMBER) ? ' colspan="2"' : ''); ?>> <td class="linecoldescription" <?php echo (! empty($conf->global->MAIN_VIEW_LINE_NUMBER) ? ' colspan="2"' : ''); ?>>
<div id="add"></div><span class="hideonsmartphone"><?php echo $langs->trans('AddNewLine'); ?></span><?php // echo $langs->trans("FreeZone"); ?> <div id="add"></div><span class="hideonsmartphone"><?php echo $langs->trans('AddNewLine'); ?></span><?php // echo $langs->trans("FreeZone"); ?>
</td> </td>
@@ -100,8 +103,10 @@ if (in_array($object->element,array('propal', 'supplier_proposal','facture','fac
?> ?>
<td class="linecoledit" colspan="<?php echo $colspan; ?>">&nbsp;</td> <td class="linecoledit" colspan="<?php echo $colspan; ?>">&nbsp;</td>
</tr> </tr>
<?php
<tr <?php echo $bcnd[$var]; ?>> }
?>
<tr class="pair nodrag nodrop nohoverpair<?php echo $nolinesbefore?'':' liste_titre_add'; ?>">
<?php <?php
if (! empty($conf->global->MAIN_VIEW_LINE_NUMBER)) { if (! empty($conf->global->MAIN_VIEW_LINE_NUMBER)) {
$coldisplay=2; } $coldisplay=2; }