aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorplegall <plg@piwigo.org>2009-12-09 22:19:06 +0000
committerplegall <plg@piwigo.org>2009-12-09 22:19:06 +0000
commit57db55318ad254446c6d3837e18bc302eb645bca (patch)
tree2a1185a34a48374451201cc4dbf2d49043c412ce /include
parent4f09b7836d49b4d0f8d51033af6c47c911705c5f (diff)
merge r4459 from branch 2.0 to trunk
feature 1316 added: we now have the ability to only fill the single value properties that are currently empty in the database instead of automatically replacing. feature 1312: as a matter of consistency, I have changed the replace_mode (that was only for multiple values properties) into multiple_value_mode + single_value_mode. git-svn-id: http://piwigo.org/svn/trunk@4460 68402e56-0260-453c-a942-63ccdbb3a9ee
Diffstat (limited to '')
-rw-r--r--include/ws_functions.inc.php55
1 files changed, 38 insertions, 17 deletions
diff --git a/include/ws_functions.inc.php b/include/ws_functions.inc.php
index c95115b3c..073c3babf 100644
--- a/include/ws_functions.inc.php
+++ b/include/ws_functions.inc.php
@@ -1665,13 +1665,6 @@ function ws_images_setInfo($params, &$service)
return new PwgError(401, 'Access denied');
}
- // name
- // category_id
- // file_content
- // file_sum
- // thumbnail_content
- // thumbnail_sum
-
$params['image_id'] = (int)$params['image_id'];
if ($params['image_id'] <= 0)
{
@@ -1691,9 +1684,7 @@ SELECT *
}
// database registration
- $update = array(
- 'id' => $params['image_id'],
- );
+ $update = array();
$info_columns = array(
'name',
@@ -1703,18 +1694,38 @@ SELECT *
'date_creation',
);
- $perform_update = false;
foreach ($info_columns as $key)
{
if (isset($params[$key]))
{
- $perform_update = true;
- $update[$key] = $params[$key];
+ if ('fill_if_empty' == $params['single_value_mode'])
+ {
+ if (empty($image_row[$key]))
+ {
+ $update[$key] = $params[$key];
+ }
+ }
+ elseif ('replace' == $params['single_value_mode'])
+ {
+ $update[$key] = $params[$key];
+ }
+ else
+ {
+ new PwgError(
+ 500,
+ '[ws_images_setInfo]'
+ .' invalid parameter single_value_mode "'.$params['single_value_mode'].'"'
+ .', possible values are {fill_if_empty, replace}.'
+ );
+ exit();
+ }
}
}
- if ($perform_update)
+ if (count(array_keys($update)) > 0)
{
+ $update['id'] = $params['image_id'];
+
include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
mass_updates(
IMAGES_TABLE,
@@ -1731,7 +1742,7 @@ SELECT *
ws_add_image_category_relations(
$params['image_id'],
$params['categories'],
- $params['replace_mode']
+ ('replace' == $params['multiple_value_mode'] ? true : false)
);
}
@@ -1742,20 +1753,30 @@ SELECT *
$tag_ids = explode(',', $params['tag_ids']);
- if ($params['replace_mode'])
+ if ('replace' == $params['multiple_value_mode'])
{
set_tags(
$tag_ids,
$params['image_id']
);
}
- else
+ elseif ('append' == $params['multiple_value_mode'])
{
add_tags(
$tag_ids,
array($params['image_id'])
);
}
+ else
+ {
+ new PwgError(
+ 500,
+ '[ws_images_setInfo]'
+ .' invalid parameter multiple_value_mode "'.$params['multiple_value_mode'].'"'
+ .', possible values are {replace, append}.'
+ );
+ exit();
+ }
}
invalidate_user_cache();