mirror/processwire
1
0
Fork 0
mirror of https://github.com/processwire/processwire.git synced 2025-08-11 09:14:58 +02:00
Code Issues Releases Wiki Activity

Update Fieldtype base class to expend methods for emptying a page field's table vs. deleting it

Browse Source
This commit is contained in:
Ryan Cramer
2021-11-05 13:20:42 -04:00
parent e56ee2e7bf
commit 495d2e1b5d
1 changed files with 24 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
wire/core/Fieldtype.php
View File

@@ -1384,24 +1384,10 @@ abstract class Fieldtype extends WireData implements Module {
* *
*/ */
public function ___deletePageField(Page $page, Field $field) { public function ___deletePageField(Page $page, Field $field) {
if(!$field->id) throw new WireException("Unable to delete from '{$field->table}' for field that doesn't exist in fields table");
// no need to delete on a new Page because it's not in the table yet // no need to delete on a new Page because it's not in the table yet
if($page->isNew()) return true; if($page->isNew()) return true;
unset($page->{$field->name}); // clear the value from the page
// clear the value from the page return $this->emptyPageFieldTable($page, $field);
// $page->set($field->name, $this->getBlankValue($page, $field));
unset($page->{$field->name});
// Delete all instances of it from the field table
$database = $this->wire('database');
$table = $database->escapeTable($field->table);
$page_id = (int) $page->id;
$query = $database->prepare("DELETE FROM `$table` WHERE pages_id=:page_id");
$query->bindValue(":page_id", $page_id, \PDO::PARAM_INT);
$result = $query->execute();
return $result;
} }
/** /**
@@ -1419,13 +1405,29 @@ abstract class Fieldtype extends WireData implements Module {
* *
*/ */
public function ___emptyPageField(Page $page, Field $field) { public function ___emptyPageField(Page $page, Field $field) {
if(!$field->id) throw new WireException("Unable to empty from '{$field->table}' for field that doesn't exist in fields table"); return $this->emptyPageFieldTable($page, $field);
$table = $this->wire('database')->escapeTable($field->table); }
$query = $this->wire('database')->prepare("DELETE FROM `$table` WHERE pages_id=:page_id");
/**
* Empty DB table of page field
*
* @param Page $page
* @param Field $field
* @return bool
* @throws WireException
* @since 3.0.189
*
*/
protected function emptyPageFieldTable(Page $page, Field $field) {
if(!$field->id) {
throw new WireException("Unable to empty table '$field->table' for field that doesn’t have an id");
}
$database = $this->wire()->database;
$table = $database->escapeTable($field->table);
$query = $database->prepare("DELETE FROM `$table` WHERE pages_id=:page_id");
$query->bindValue(":page_id", $page->id, \PDO::PARAM_INT); $query->bindValue(":page_id", $page->id, \PDO::PARAM_INT);
return $query->execute(); return $query->execute();
} }
/** /**
* Move this field’s data from one page to another. * Move this field’s data from one page to another.
@@ -1441,7 +1443,7 @@ abstract class Fieldtype extends WireData implements Module {
public function ___replacePageField(Page $src, Page $dst, Field $field) { public function ___replacePageField(Page $src, Page $dst, Field $field) {
$database = $this->wire()->database; $database = $this->wire()->database;
$table = $database->escapeTable($field->table); $table = $database->escapeTable($field->table);
$this->emptyPageField($dst, $field); $this->deletePageField($dst, $field);
// move the data // move the data
$sql = "UPDATE `$table` SET pages_id=:dstID WHERE pages_id=:srcID"; $sql = "UPDATE `$table` SET pages_id=:dstID WHERE pages_id=:srcID";
$query = $database->prepare($sql); $query = $database->prepare($sql);