Skip to content

Commit dc932bd

Browse files
committed
API Stop using deprecated API
1 parent 8822b0f commit dc932bd

2 files changed

Lines changed: 14 additions & 3 deletions

File tree

src/StringTagField.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
use SilverStripe\Control\Controller;
77
use SilverStripe\Control\HTTPRequest;
88
use SilverStripe\Control\HTTPResponse;
9+
use SilverStripe\Dev\Deprecation;
910
use SilverStripe\Forms\DropdownField;
1011
use SilverStripe\Forms\Validator;
1112
use SilverStripe\ORM\ArrayList;
@@ -281,7 +282,10 @@ public function saveInto(DataObjectInterface $record)
281282

282283
$record->$name = $this->dataValue();
283284

284-
if (self::config()->get('immediate_write_enabled')) {
285+
$immediate_write_enabled = Deprecation::withNoReplacement(function () {
286+
return self::config()->get('immediate_write_enabled');
287+
});
288+
if ($immediate_write_enabled) {
285289
$record->write();
286290
}
287291
}

tests/StringTagFieldTest.php

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
use SilverStripe\Forms\Form;
99
use SilverStripe\TagField\StringTagField;
1010
use SilverStripe\TagField\Tests\Stub\StringTagFieldTestBlogPost;
11+
use SilverStripe\Dev\Deprecation;
1112

1213
class StringTagFieldTest extends SapphireTest
1314
{
@@ -63,10 +64,13 @@ public function testItSavesTagsOnExistingRecords()
6364

6465
public function testImmediateWriteEnabled()
6566
{
67+
if (Deprecation::isEnabled()) {
68+
$this->markTestSkipped('Test calls deprecated code');
69+
}
6670
$record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
6771
$record->write();
6872

69-
StringTagField::config()->update('immediate_write_enabled', true);
73+
StringTagField::config()->set('immediate_write_enabled', true);
7074

7175
$field = new StringTagField('Tags');
7276
$field->setValue(['Tag1', 'Tag2']);
@@ -77,10 +81,13 @@ public function testImmediateWriteEnabled()
7781

7882
public function testImmediateWriteDisabled()
7983
{
84+
if (Deprecation::isEnabled()) {
85+
$this->markTestSkipped('Test calls deprecated code');
86+
}
8087
$record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
8188
$record->write();
8289

83-
StringTagField::config()->update('immediate_write_enabled', false);
90+
StringTagField::config()->set('immediate_write_enabled', false);
8491

8592
$field = new StringTagField('Tags');
8693
$field->setValue(['Tag1', 'Tag2']);

0 commit comments

Comments
 (0)