From ca2dde9a41f967d672d8ad780dc37f247928dac9 Mon Sep 17 00:00:00 2001 From: Diogo Cordeiro Date: Sun, 3 Nov 2019 16:49:10 +0000 Subject: [PATCH] [ExtendedProfile] Fix updates and allow to delete values --- classes/Managed_DataObject.php | 16 ++-- .../ExtendedProfile/ExtendedProfilePlugin.php | 2 +- .../actions/profiledetailsettings.php | 91 ++++++++----------- .../classes/Profile_detail.php | 52 +++++------ 4 files changed, 72 insertions(+), 89 deletions(-) diff --git a/classes/Managed_DataObject.php b/classes/Managed_DataObject.php index 271205eebd..407363d709 100644 --- a/classes/Managed_DataObject.php +++ b/classes/Managed_DataObject.php @@ -522,14 +522,18 @@ abstract class Managed_DataObject extends Memcached_DataObject return $aliases; } - // 'update' won't write key columns, so we have to do it ourselves. - // This also automatically calls "update" _before_ it sets the keys. - // FIXME: This only works with single-column primary keys so far! Beware! /** - * @param DB_DataObject &$orig Must be "instanceof" $this - * @param string $pid Primary ID column (no escaping is done on column name!) + * update() won't write key columns, so we have to do it ourselves. + * This also automatically calls "update" _before_ it sets the keys. + * FIXME: This only works with single-column primary keys so far! Beware! + * + * @param Managed_DataObject $orig Must be "instanceof" $this + * @param string $pid Primary ID column (no escaping is done on column name!) + * @return bool|void + * @throws MethodNotImplementedException + * @throws ServerException */ - public function updateWithKeys(Managed_DataObject $orig, $pid=null) + public function updateWithKeys(Managed_DataObject $orig, ?string $pid = null) { if (!$orig instanceof $this) { throw new ServerException('Tried updating a DataObject with a different class than itself.'); diff --git a/plugins/ExtendedProfile/ExtendedProfilePlugin.php b/plugins/ExtendedProfile/ExtendedProfilePlugin.php index c3b7492d2c..6f1571e9cb 100644 --- a/plugins/ExtendedProfile/ExtendedProfilePlugin.php +++ b/plugins/ExtendedProfile/ExtendedProfilePlugin.php @@ -33,7 +33,7 @@ include_once __DIR__ . '/lib/profiletools.php'; class ExtendedProfilePlugin extends Plugin { - const PLUGIN_VERSION = '3.0.1'; + const PLUGIN_VERSION = '3.0.2'; public function onPluginVersion(array &$versions): bool { diff --git a/plugins/ExtendedProfile/actions/profiledetailsettings.php b/plugins/ExtendedProfile/actions/profiledetailsettings.php index f0f3b522a1..d1aa5dc165 100644 --- a/plugins/ExtendedProfile/actions/profiledetailsettings.php +++ b/plugins/ExtendedProfile/actions/profiledetailsettings.php @@ -35,20 +35,6 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction return _m('Extended profile settings'); } - public function showStylesheets() - { - parent::showStylesheets(); - $this->cssLink('plugins/ExtendedProfile/css/profiledetail.css'); - return true; - } - - public function showScripts() - { - parent::showScripts(); - $this->script('plugins/ExtendedProfile/js/profiledetail.js'); - return true; - } - protected function doPost() { if ($this->arg('save')) { @@ -75,26 +61,18 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction $this->saveStandardProfileDetails(); - $simpleFieldNames = array('title', 'spouse', 'kids', 'manager'); - $dateFieldNames = array('birthday'); + $simpleFieldNames = ['title', 'spouse', 'kids', 'manager']; + $dateFieldNames = ['birthday']; foreach ($simpleFieldNames as $name) { $value = $this->trimmed('extprofile-' . $name); - if (!empty($value)) { - $this->saveField($name, $value); - } + $this->saveField($name, $value); } foreach ($dateFieldNames as $name) { $value = $this->trimmed('extprofile-' . $name); - $dateVal = $this->parseDate($name, $value); - $this->saveField( - $name, - null, - null, - null, - $dateVal - ); + $date_val = $this->parseDate($name, $value); + $this->saveField($name, null, null, null, $date_val); } $this->savePhoneNumbers(); @@ -140,6 +118,7 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction public function savePhoneNumbers() { + common_debug('save phone numbers'); $phones = $this->findPhoneNumbers(); $this->removeAll('phone'); $i = 0; @@ -164,17 +143,17 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction // 'extprofile-phone-1-rel' => 'mobile', $phones = $this->sliceParams('phone', 2); - $phoneArray = array(); + $phone_array = []; foreach ($phones as $phone) { list($number, $rel) = array_values($phone); - $phoneArray[] = array( + $phone_array[] = [ 'value' => $number, 'rel' => $rel - ); + ]; } - return $phoneArray; + return $phone_array; } public function findIms() @@ -189,10 +168,10 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction foreach ($ims as $im) { list($id, $rel) = array_values($im); - $imArray[] = array( + $imArray[] = [ 'value' => $id, 'rel' => $rel - ); + ]; } return $imArray; @@ -200,6 +179,7 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction public function saveIms() { + common_debug('save ims'); $ims = $this->findIms(); $this->removeAll('im'); $i = 0; @@ -237,6 +217,7 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction public function saveWebsites() { + common_debug('save websites'); $sites = $this->findWebsites(); $this->removeAll('website'); $i = 0; @@ -272,11 +253,11 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction $expArray = array(); foreach ($experiences as $exp) { - if (sizeof($experiences) == 4) { - list($company, $current, $end, $start) = array_values($exp); + if (count($exp) === 4) { + [$company, $current, $end, $start] = array_values($exp); } else { $end = null; - list($company, $current, $start) = array_values($exp); + [$company, $current, $start] = array_values($exp); } if (!empty($company)) { $expArray[] = array( @@ -372,7 +353,7 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction public function saveEducations() { - common_debug('save education'); + common_debug('save educations'); $edus = $this->findEducations(); common_debug(var_export($edus, true)); @@ -469,39 +450,38 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction */ public function saveField(string $name, ?string $value, ?string $rel = null, ?int $index = null, ?string $date = null) { + common_debug('save ' . $name); $detail = new Profile_detail(); $detail->profile_id = $this->scoped->getID(); $detail->field_name = $name; $detail->value_index = $index; - $result = $detail->find(true); + if ($detail->find(true)) { + // Found an existing Profile Detail, let's update it! + $orig = clone($detail); - if (!$result instanceof Profile_detail) { - $detail->value_index = $index; + $detail->field_value = $value ?? $detail->sqlValue('NULL'); + $detail->rel = $rel ?? $detail->sqlValue('NULL'); + $detail->date = $date ?? $detail->sqlValue('NULL'); + + if (!$detail->update($orig)) { + common_log_db_error($detail, 'UPDATE', __FILE__); + // TRANS: Server error displayed when a field could not be saved in the database. + throw new ServerException(_m('Could not save profile details.')); + } + } elseif (!empty($rel) || !empty($value) || !empty($date)) { + // It's the first time, let's insert, if there is anything to insert. $detail->rel = $rel; $detail->field_value = $value; $detail->date = $date; $detail->created = common_sql_now(); - $result = $detail->insert(); - if ($result === false) { + + if (!$detail->insert()) { common_log_db_error($detail, 'INSERT', __FILE__); // TRANS: Server error displayed when a field could not be saved in the database. throw new ServerException(_m('Could not save profile details.')); } - } else { - $orig = clone($detail); - - $detail->field_value = $value; - $detail->rel = $rel; - $detail->date = $date; - - $result = $detail->update($orig); - if ($result === false) { - common_log_db_error($detail, 'UPDATE', __FILE__); - // TRANS: Server error displayed when a field could not be saved in the database. - throw new ServerException(_m('Could not save profile details.')); - } } $detail->free(); @@ -593,6 +573,7 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction private function saveCustomFields($action) { + common_debug('save custom fields'); $fields = GNUsocialProfileExtensionField::allFields(); $user = common_current_user(); $profile = $user->getProfile(); diff --git a/plugins/ExtendedProfile/classes/Profile_detail.php b/plugins/ExtendedProfile/classes/Profile_detail.php index a186b626f5..a766f20ac2 100644 --- a/plugins/ExtendedProfile/classes/Profile_detail.php +++ b/plugins/ExtendedProfile/classes/Profile_detail.php @@ -71,37 +71,35 @@ class Profile_detail extends Managed_DataObject public static function schemaDef() { - return array( + return [ // No need for i18n. Table properties. - 'description' - => 'Additional profile details for the ExtendedProfile plugin', - 'fields' => array( - 'id' => array('type' => 'serial', 'not null' => true), - 'profile_id' => array('type' => 'int', 'not null' => true), - 'field_name' => array( + 'description' => 'Additional profile details for the ExtendedProfile plugin', + 'fields' => [ + 'id' => ['type' => 'serial', 'not null' => true], + 'profile_id' => ['type' => 'int', 'not null' => true], + 'field_name' => [ 'type' => 'varchar', 'length' => 16, - 'not null' => true - ), - 'value_index' => array('type' => 'int'), - 'field_value' => array('type' => 'text'), - 'date' => array('type' => 'datetime'), - 'rel' => array('type' => 'varchar', 'length' => 16), - 'rel_profile' => array('type' => 'int'), - 'created' => array( + 'not null' => true, + ], + 'value_index' => ['type' => 'int'], + 'field_value' => ['type' => 'text'], + 'date' => ['type' => 'datetime'], + 'rel' => ['type' => 'varchar', 'length' => 16], + 'rel_profile' => ['type' => 'int'], + 'created' => [ 'type' => 'datetime', - 'not null' => true - ), - 'modified' => array( + 'not null' => true, + ], + 'modified' => [ 'type' => 'timestamp', - 'not null' => true - ), - ), - 'primary key' => array('id'), - 'unique keys' => array( - 'profile_detail_profile_id_field_name_value_index' - => array('profile_id', 'field_name', 'value_index'), - ) - ); + 'not null' => true, + ], + ], + 'primary key' => ['id'], + 'unique keys' => [ + 'profile_detail_profile_id_field_name_value_index' => ['profile_id', 'field_name', 'value_index'] + ], + ]; } }