Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/3.8' into develop
Browse files Browse the repository at this point in the history
Conflicts:
	htdocs/core/lib/memory.lib.php
  • Loading branch information
eldy committed Sep 8, 2015
2 parents 0a6451d + 82ad81f commit 24f7100
Show file tree
Hide file tree
Showing 9 changed files with 35 additions and 41 deletions.
10 changes: 5 additions & 5 deletions dev/skeletons/build_class_from_table.php
Expand Up @@ -69,8 +69,9 @@
}

// Show parameters
print 'Tablename='.$argv[1]."\n";
print "Current dir is ".getcwd()."\n";
print 'Tablename: '.$argv[1]."\n";
print "Current dir: ".getcwd()."\n";
print "Database name: ".$db->database_name."\n";


// Define array with list of properties
Expand Down Expand Up @@ -304,10 +305,9 @@
}
else
{
$varprop.=' \'.(! isset($this->'.$prop['field'].')?\'NULL\':"\'".';
$varprop.='\'.(! isset($this->'.$prop['field'].')?\'NULL\':"\'".';
$varprop.="\$this->".$prop['field']."";
$varprop.='.\').';

$varprop.='"\'")';
}

if ($i < (count($property)-$no_output_field)) $varprop.=".','";
Expand Down
4 changes: 2 additions & 2 deletions dev/skeletons/modMyModule.class.php
Expand Up @@ -193,7 +193,7 @@ public function __construct($db)
// Add here entries to declare new menus
//
// Example to declare a new Top Menu entry and its Left menu entry:
// $this->menu[$r]=array( 'fk_menu'=>0, // Put 0 if this is a top menu
// $this->menu[$r]=array( 'fk_menu'=>'', // '' if this is a top menu. For left menu, use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
// 'type'=>'top', // This is a Top menu entry
// 'titre'=>'MyModule top menu',
// 'mainmenu'=>'mymodule',
Expand All @@ -208,7 +208,7 @@ public function __construct($db)
// $r++;
//
// Example to declare a Left Menu entry into an existing Top menu entry:
// $this->menu[$r]=array( 'fk_menu'=>'fk_mainmenu=xxx', // Use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
// $this->menu[$r]=array( 'fk_menu'=>'fk_mainmenu=xxx', // '' if this is a top menu. For left menu, use 'fk_mainmenu=xxx' or 'fk_mainmenu=xxx,fk_leftmenu=yyy' where xxx is mainmenucode and yyy is a leftmenucode
// 'type'=>'left', // This is a Left menu entry
// 'titre'=>'MyModule left menu',
// 'mainmenu'=>'xxx',
Expand Down
5 changes: 0 additions & 5 deletions dev/skeletons/skeleton_class.class.php
Expand Up @@ -38,11 +38,6 @@
*/
class Skeleton_Class extends CommonObject
{
/**
* @var DoliDb Database handler
*/
protected $db;

/**
* @var string Error code (or message)
* @deprecated
Expand Down
10 changes: 5 additions & 5 deletions dev/skeletons/skeleton_page.php
Expand Up @@ -43,7 +43,7 @@
if (! $res && file_exists("../../../../dolibarr/htdocs/main.inc.php")) $res=@include '../../../../dolibarr/htdocs/main.inc.php'; // Used on dev env only
if (! $res) die("Include of main fails");
// Change this following line to use the correct relative path from htdocs
include_once(DOL_DOCUMENT_ROOT.'/core/class/formcompany.class.php');
include_once(DOL_DOCUMENT_ROOT.'/core/class/html.formcompany.class.php');
dol_include_once('/module/class/skeleton_class.class.php');

// Load traductions files requiredby by page
Expand All @@ -62,7 +62,7 @@
//accessforbidden();
}

if (empty($action) && empty($id) && empty($ref)) $action='create';
if (empty($action) && empty($id) && empty($ref)) $action='list';

// Load object if id or ref is provided as parameter
$object=new Skeleton_Class($db);
Expand Down Expand Up @@ -225,7 +225,7 @@ function init_myfunc()


// Part to show a list
if ($action == 'list' || empty($id))
if ($action == 'list' || (empty($id) && $action != 'create'))
{
$sql = "SELECT";
$sql.= " t.rowid,";
Expand All @@ -243,7 +243,7 @@ function init_myfunc()
$parameters=array();
$reshook=$hookmanager->executeHooks('printFieldListWhere',$parameters); // Note that $action and $object may have been modified by hook
$sql.=$hookmanager->resPrint;
$sql.= " ORDER BY field1 ASC";
$sql.=$db->order($sortfield, $sortorder);

print '<form method="GET" id="searchFormList" action="'.$_SERVER["PHP_SELF"].'">';

Expand Down Expand Up @@ -329,7 +329,7 @@ function init_myfunc()
// Part to create
if ($action == 'create')
{
print_fiche_titre($langs->trans("NewResidence"));
print_fiche_titre($langs->trans("NewSkeleton"));

print '<form method="POST" action="'.$_SERVER["PHP_SELF"].'">';
print '<input type="hidden" name="action" value="add">';
Expand Down
2 changes: 1 addition & 1 deletion htdocs/contact/card.php
Expand Up @@ -646,7 +646,7 @@
print '<td colspan="2"><label for="birthday_alert">'.$langs->trans("Alert").'</label>: ';
if ($object->birthday_alert)
{
print '<input type="checkbox" name="birthday_alert" id="birthday_aler" checked></td>';
print '<input type="checkbox" name="birthday_alert" id="birthday_alert" checked></td>';
}
else
{
Expand Down
10 changes: 5 additions & 5 deletions htdocs/core/class/commonobject.class.php
Expand Up @@ -53,6 +53,11 @@ abstract class CommonObject
*/
public $error;

/**
* @var string[] Array of error strings
*/
public $errors=array();

/**
* @var string Key value used to track if data is coming from import wizard
*/
Expand Down Expand Up @@ -82,11 +87,6 @@ abstract class CommonObject

// Following vars are used by some objects only. We keep this property here in CommonObject to be able to provide common method using them.

/**
* @var string[] Array of error strings
*/
public $errors=array();

/**
* @var string[] Can be used to pass information when only object is provided to method
*/
Expand Down
2 changes: 1 addition & 1 deletion htdocs/core/lib/functions.lib.php
Expand Up @@ -2766,7 +2766,7 @@ function print_fiche_titre($title, $mesg='', $picto='title_generic.png', $pictoi
* @param int $id To force an id on html objects
* @return string
*/
function load_fiche_titre($titre, $mesg='', $picto='title.png', $pictoisfullpath=0, $id=0)
function load_fiche_titre($titre, $mesg='', $picto='title_generic.png', $pictoisfullpath=0, $id=0)
{
global $conf;

Expand Down
27 changes: 13 additions & 14 deletions htdocs/core/lib/memory.lib.php
Expand Up @@ -47,20 +47,19 @@ function dol_setcache($memoryid,$data)
// Using a memcached server
if (! empty($conf->memcached->enabled) && class_exists('Memcached'))
{
global $m;
if (empty($m) || ! is_object($m))
global $dolmemcache;
if (empty($dolmemcache) || ! is_object($dolmemcache))
{
$m=new Memcached();
$dolmemcache=new Memcached();
$tmparray=explode(':',$conf->global->MEMCACHED_SERVER);
$result=$m->addServer($tmparray[0], $tmparray[1]?$tmparray[1]:11211);
$result=$dolmemcache->addServer($tmparray[0], $tmparray[1]?$tmparray[1]:11211);
if (! $result) return -1;
}

$memoryid=session_name().'_'.$memoryid;
//$m->setOption(Memcached::OPT_COMPRESSION, false);
//print "Add memoryid=".$memoryid;
$m->add($memoryid,$data); // This fails if key already exists
$rescode=$m->getResultCode();
//$dolmemcache->setOption(Memcached::OPT_COMPRESSION, false);
$dolmemcache->add($memoryid,$data); // This fails if key already exists
$rescode=$dolmemcache->getResultCode();
if ($rescode == 0)
{
return count($data);
Expand All @@ -72,18 +71,18 @@ function dol_setcache($memoryid,$data)
}
else if (! empty($conf->memcached->enabled) && class_exists('Memcache'))
{
global $m;
if (empty($m) || ! is_object($m))
global $dolmemcache;
if (empty($dolmemcache) || ! is_object($dolmemcache))
{
$m=new Memcache();
$dolmemcache=new Memcache();
$tmparray=explode(':',$conf->global->MEMCACHED_SERVER);
$result=$m->addServer($tmparray[0], $tmparray[1]?$tmparray[1]:11211);
$result=$dolmemcache->addServer($tmparray[0], $tmparray[1]?$tmparray[1]:11211);
if (! $result) return -1;
}

$memoryid=session_name().'_'.$memoryid;
//$m->setOption(Memcached::OPT_COMPRESSION, false);
$result=$m->add($memoryid,$data); // This fails if key already exists
//$dolmemcache->setOption(Memcached::OPT_COMPRESSION, false);
$result=$dolmemcache->add($memoryid,$data); // This fails if key already exists
if ($result)
{
return count($data);
Expand Down
6 changes: 3 additions & 3 deletions htdocs/opensurvey/card.php
Expand Up @@ -248,7 +248,7 @@
print '<tr><td>'.$langs->trans('ToReceiveEMailForEachVote').'</td><td colspan="2">';
if ($action == 'edit')
{
print '<input type="checkbox" name="mailsonde" size="40"'.($object->mailsonde?' checked':'').'">';
print '<input type="checkbox" name="mailsonde" '.($object->mailsonde?'checked="checked"':'').'">';
}
else {
print yn($object->mailsonde);
Expand All @@ -266,7 +266,7 @@
print '<tr><td>'.$langs->trans('CanComment').'</td><td colspan="2">';
if ($action == 'edit')
{
print '<input type="checkbox" name="cancomment" size="40"'.($object->allow_comments?' checked':'').'">';
print '<input type="checkbox" name="cancomment" '.($object->allow_comments?'checked="checked"':'').'">';
}
else print yn($object->allow_comments);
print '</td></tr>';
Expand All @@ -275,7 +275,7 @@
print '<tr><td>'.$langs->trans('CanSeeOthersVote').'</td><td colspan="2">';
if ($action == 'edit')
{
print '<input type="checkbox" name="canseeothersvote" size="40"'.($object->allow_spy?' checked':'').'">';
print '<input type="checkbox" name="canseeothersvote" '.($object->allow_spy?'checked="checked"':'').'">';
}
else print yn($object->allow_spy);
print '</td></tr>';
Expand Down

0 comments on commit 24f7100

Please sign in to comment.