Honor item delivery data legacy fields

This commit is contained in:
Hypolite Petovan 2018-12-08 15:22:54 -05:00
parent 85eb6affdf
commit 38734f17b9
2 changed files with 6 additions and 3 deletions

View file

@ -551,7 +551,7 @@ class Item extends BaseObject
$fields['item-content'] = array_merge(self::CONTENT_FIELDLIST, self::MIXED_CONTENT_FIELDLIST); $fields['item-content'] = array_merge(self::CONTENT_FIELDLIST, self::MIXED_CONTENT_FIELDLIST);
$fields['item-delivery-data'] = ItemDeliveryData::FIELD_LIST; $fields['item-delivery-data'] = array_merge(ItemDeliveryData::LEGACY_FIELD_LIST, ItemDeliveryData::FIELD_LIST);
$fields['permissionset'] = ['allow_cid', 'allow_gid', 'deny_cid', 'deny_gid']; $fields['permissionset'] = ['allow_cid', 'allow_gid', 'deny_cid', 'deny_gid'];
@ -730,7 +730,7 @@ class Item extends BaseObject
foreach ($fields as $table => $table_fields) { foreach ($fields as $table => $table_fields) {
foreach ($table_fields as $field => $select) { foreach ($table_fields as $field => $select) {
if (empty($selected) || in_array($select, $selected)) { if (empty($selected) || in_array($select, $selected)) {
$legacy_fields = array_merge(ItemDeliveryData::FIELD_LIST, self::MIXED_CONTENT_FIELDLIST); $legacy_fields = array_merge(ItemDeliveryData::LEGACY_FIELD_LIST, self::MIXED_CONTENT_FIELDLIST);
if (self::isLegacyMode() && in_array($select, $legacy_fields)) { if (self::isLegacyMode() && in_array($select, $legacy_fields)) {
$selection[] = "`item`.`".$select."` AS `internal-item-" . $select . "`"; $selection[] = "`item`.`".$select."` AS `internal-item-" . $select . "`";
} }

View file

@ -10,10 +10,13 @@ use Friendica\Database\DBA;
class ItemDeliveryData class ItemDeliveryData
{ {
const FIELD_LIST = [ const LEGACY_FIELD_LIST = [
// Legacy fields moved from item table // Legacy fields moved from item table
'postopts', 'postopts',
'inform', 'inform',
];
const FIELD_LIST = [
// New delivery fields with virtual field name in item fields // New delivery fields with virtual field name in item fields
'queue_count' => 'delivery_queue_count', 'queue_count' => 'delivery_queue_count',
'queue_done' => 'delivery_queue_done', 'queue_done' => 'delivery_queue_done',