Skip to content

Commit

Permalink
Work on inventory module debug
Browse files Browse the repository at this point in the history
  • Loading branch information
@eldy
eldy committed May 6, 2017
1 parent f4447f9 commit 46ff62f
Show file tree
Hide file tree
Showing 10 changed files with 79 additions and 57 deletions.
2 changes: 1 addition & 1 deletion htdocs/admin/stock.php
View file
Expand Up @@ -434,7 +434,7 @@


print '<br />';
if ($conf->global->MAIN_LEVEL_FEATURES >= 2)
if ($conf->global->MAIN_FEATURES_LEVEL >= 2)
{
$var=false;
print '<table class="noborder" width="100%">';
Expand Down
34 changes: 21 additions & 13 deletions htdocs/core/class/commonobject.class.php
View file
Expand Up @@ -4938,25 +4938,33 @@ public function fetchCommon($id, $ref = null)

if(!empty($id)) $sql.= ' WHERE rowid = '.$id;
else $sql.= ' WHERE ref = \''.$this->quote($ref).'\'';

$res = $this->db->query($sql);
if($obj = $this->db->fetch_object($res))
if ($res)
{
$this->id = $id;
$this->set_vars_by_db($obj);

$this->datec = $this->db->idate($obj->datec);
$this->tms = $this->db->idate($obj->tms);

return $this->id;
if ($obj = $this->db->fetch_object($res))
{
$this->id = $id;
$this->set_vars_by_db($obj);

$this->datec = $this->db->idate($obj->datec);
$this->tms = $this->db->idate($obj->tms);

return $this->id;
}
else
{
$this->error = $this->db->lasterror();
$this->errors[] = $this->error;
return -1;
}
}
else
{
$this->error = $this->db->lasterror();
$this->errors[] = $this->error;
return -1;
$this->error = $this->db->lasterror();
$this->errors[] = $this->error;
return -1;
}

}

/**
Expand Down
2 changes: 0 additions & 2 deletions htdocs/core/class/coreobject.class.php
View file
Expand Up @@ -106,14 +106,12 @@ private function checkFieldType($field, $type)
*/
public function fetch($id, $loadChild = true)
{

$res = $this->fetchCommon($id);
if($res>0) {
if ($loadChild) $this->fetchChild();
}

return $res;

}


Expand Down
2 changes: 1 addition & 1 deletion htdocs/core/menus/standard/eldy.lib.php
View file
Expand Up @@ -1265,7 +1265,7 @@ function print_left_eldy_menu($db,$menu_array_before,$menu_array_after,&$tabMenu
}

// Inventory
if ($conf->global->MAIN_LEVEL_FEATURES >= 2)
if ($conf->global->MAIN_FEATURES_LEVEL >= 2)
{
if (! empty($conf->stock->enabled))
{
Expand Down
35 changes: 20 additions & 15 deletions htdocs/core/modules/modStock.class.php
View file
Expand Up @@ -117,32 +117,37 @@ function __construct($db)
$this->rights[4][4] = 'mouvement';
$this->rights[4][5] = 'creer';

if ($conf->global->MAIN_LEVEL_FEATURES >= 2) {
if ($conf->global->MAIN_FEATURES_LEVEL >= 2) {

$this->rights[5][0] = 1006;
$this->rights[5][0] = 1011;
$this->rights[5][1] = 'inventoryReadPermission'; // Permission label
$this->rights[5][3] = 0; // Permission by default for new user (0/1)
$this->rights[5][4] = 'read'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[5][4] = 'advance_inventory'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[5][5] = 'read'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)

$this->rights[6][0] = 1007;
$this->rights[6][0] = 1012;
$this->rights[6][1] = 'inventoryCreatePermission'; // Permission label
$this->rights[6][3] = 0; // Permission by default for new user (0/1)
$this->rights[6][4] = 'create'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[6][4] = 'advance_inventory'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[6][5] = 'create'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)

$this->rights[7][0] = 1008;
$this->rights[7][0] = 1013;
$this->rights[7][1] = 'inventoryWritePermission'; // Permission label
$this->rights[7][3] = 0; // Permission by default for new user (0/1)
$this->rights[7][4] = 'write'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[7][4] = 'advance_inventory'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[7][5] = 'write'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)

$this->rights[7][0] = 1009;
$this->rights[7][1] = 'inventoryValidatePermission'; // Permission label
$this->rights[7][3] = 0; // Permission by default for new user (0/1)
$this->rights[7][4] = 'validate'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[8][0] = 1014;
$this->rights[8][1] = 'inventoryValidatePermission'; // Permission label
$this->rights[8][3] = 0; // Permission by default for new user (0/1)
$this->rights[8][4] = 'advance_inventory'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[8][5] = 'validate'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)

$this->rights[7][0] = 1010;
$this->rights[7][1] = 'inventoryChangePMPPermission'; // Permission label
$this->rights[7][3] = 0; // Permission by default for new user (0/1)
$this->rights[7][4] = 'changePMP'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[9][0] = 1015;
$this->rights[9][1] = 'inventoryChangePMPPermission'; // Permission label
$this->rights[9][3] = 0; // Permission by default for new user (0/1)
$this->rights[9][4] = 'advance_inventory'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)
$this->rights[9][5] = 'changePMP'; // In php code, permission will be checked by test if ($user->rights->permkey->level1->level2)

}

Expand Down
1 change: 0 additions & 1 deletion htdocs/install/mysql/tables/llx_fichinter.sql
View file
Expand Up @@ -43,5 +43,4 @@ create table llx_fichinter
note_public text,
model_pdf varchar(255),
extraparams varchar(255) -- for stock other parameters with json format

)ENGINE=innodb;
25 changes: 15 additions & 10 deletions htdocs/install/mysql/tables/llx_inventory.sql
View file
@@ -1,6 +1,6 @@
-- ===================================================================
-- Copyright (C) 2012 Laurent Destailleur <eldy@users.sourceforge.net>
-- Copyright (C) 2017 ATM Consulting <support@atm-consulting.fr>
-- Copyright (C) 2017 Laurent Destailleur <eldy@users.sourceforge.net>
-- Copyright (C) 2017 ATM Consulting <support@atm-consulting.fr>
--
-- This program is free software; you can redistribute it and/or modify
-- it under the terms of the GNU General Public License as published by
Expand All @@ -19,13 +19,18 @@

CREATE TABLE llx_inventory
(
rowid integer NOT NULL AUTO_INCREMENT PRIMARY KEY,
datec datetime DEFAULT NULL,
tms timestamp,
fk_warehouse integer DEFAULT 0,
entity integer DEFAULT 0,
status integer DEFAULT 0,
title varchar(255) NOT NULL,
date_inventory datetime DEFAULT NULL
rowid integer NOT NULL AUTO_INCREMENT PRIMARY KEY,
entity integer DEFAULT 0,
ref varchar(48),
datec datetime DEFAULT NULL,
tms timestamp,
fk_user_author integer, -- user making creation
fk_user_modif integer, -- user making last change
fk_user_valid integer, -- valideur de la fiche
fk_warehouse integer DEFAULT 0,
status integer DEFAULT 0,
title varchar(255) NOT NULL,
date_inventory datetime DEFAULT NULL,
import_key varchar(14) -- import key
)
ENGINE=InnoDB;
27 changes: 16 additions & 11 deletions htdocs/product/inventory/card.php
View file
Expand Up @@ -361,9 +361,9 @@
dol_fiche_end();

print '<div class="center">';
print '<input type="submit" class="button" value="'.$langs->trans('inventoryConfirmCreate').'" />';
print '<input type="submit" class="button" name="create" value="'.$langs->trans('inventoryConfirmCreate').'" />';
print ' &nbsp; &nbsp; ';
print '<input type="submit" class="button" value="'.$langs->trans('inventoryConfirmCreate').'" />';
print '<input type="submit" class="button" name="cancel" value="'.$langs->trans('Cancel').'" />';
print '</div>';

echo '</form>';
Expand All @@ -373,8 +373,14 @@
if ($action == 'view' || $action == 'edit' || empty($action))
{
$object = new Inventory($db);
$object->fetch($id);

$result = $object->fetch($id);
if ($result < 0) dol_print_error($db, $object->error, $object->errors);

$warehouse = new Entrepot($db);
$warehouse->fetch($object->fk_warehouse);



if($action == 'changePMP')
{
print $form->formconfirm($_SERVER["PHP_SELF"].'?id='.$object->id, $langs->trans('ApplyNewPMP'), $langs->trans('ConfirmApplyNewPMP', $object->getTitle()), 'confirm_changePMP', array(),'no',1);
Expand All @@ -396,18 +402,17 @@
print $form->formconfirm($_SERVER["PHP_SELF"].'?id='.$object->id,$langs->trans('RegulateStock'),$langs->trans('ConfirmRegulateStock',$object->getTitle()),'confirm_regulate',array(),'no',1);
}

$warehouse = new Entrepot($db);
$warehouse->fetch($object->fk_warehouse);

print dol_get_fiche_head(inventoryPrepareHead($object, $langs->trans('inventoryOfWarehouse', $warehouse->libelle), empty($action) ? '': '&action='.$action));

$lines = array();
card_line($object, $lines, $action);

print '<b>'.$langs->trans('inventoryOnDate')." ".$object->getDate('date_inventory').'</b><br><br>';
print $langs->trans('Ref')." ".$object->ref.'<br>';
print $langs->trans('Date')." ".$object->getDate('date_inventory').'<br><br>';

$objectTPL = array(
'id'=> $object->id
'id'=> $object->id
,'ref'=> $object->ref
,'date_cre' => $object->getDate('date_cre', 'd/m/Y')
,'date_maj' => $object->getDate('date_maj', 'd/m/Y H:i')
,'fk_warehouse' => $object->fk_warehouse
Expand Down Expand Up @@ -608,8 +613,8 @@ function _headerList($view)

?>
<tr style="background-color:#dedede;">
<th align="left" width="20%">&nbsp;&nbsp;Produit</th>
<th align="center"><?php echo $langs->trans('Warehouse'); ?></th>
<th class="titlefield"><?php echo $langs->trans('Product'); ?></th>
<th><?php echo $langs->trans('Warehouse'); ?></th>
<?php if (! empty($conf->barcode->enabled)) { ?>
<th align="center"><?php echo $langs->trans('Barcode'); ?></th>
<?php } ?>
Expand Down
4 changes: 3 additions & 1 deletion htdocs/product/inventory/class/inventory.class.php
View file
Expand Up @@ -69,6 +69,7 @@ class Inventory extends CoreObject
*/
protected $fields=array(
'fk_warehouse'=>array('type'=>'integer','index'=>true)
,'ref'=>array('type'=>'string','index'=>true)
,'entity'=>array('type'=>'integer','index'=>true)
,'status'=>array('type'=>'integer','index'=>true)
,'date_inventory'=>array('type'=>'date')
Expand Down Expand Up @@ -113,7 +114,8 @@ public function fetch($id, $loadChild = true)
if(!$loadChild) $this->withChild = false;

$res = parent::fetch($id, $loadChild);
if($res > 0)

if ($res > 0)
{
$this->sortDet();
$this->amount = 0;
Expand Down
4 changes: 2 additions & 2 deletions htdocs/product/inventory/tpl/inventory.tpl.php
View file
Expand Up @@ -80,7 +80,7 @@ function hide_save_button() {
<?php echo inventorySelectProducts($object); ?>

<input class="button" type="submit" value="<?php echo $langs->trans('AddProduct'); ?>" />
</form>
</form><br>
<?php } ?>

<form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
Expand All @@ -92,7 +92,7 @@ function hide_save_button() {
<input type="hidden" name="action" value="save" />
<input type="hidden" name="id" value="<?php echo $object->id; ?>" />

<table width="100%" class="border workstation">
<table width="100%" class="noborder workstation">
<?php

_headerList($view);
Expand Down

0 comments on commit 46ff62f

Please sign in to comment.