Fix updatePageRule() method
This commit is contained in:
@@ -111,6 +111,7 @@ class PageRules implements API
|
|||||||
|
|
||||||
public function updatePageRule(
|
public function updatePageRule(
|
||||||
string $zoneID,
|
string $zoneID,
|
||||||
|
string $ruleID,
|
||||||
PageRulesTargets $target = null,
|
PageRulesTargets $target = null,
|
||||||
PageRulesActions $actions = null,
|
PageRulesActions $actions = null,
|
||||||
bool $active = null,
|
bool $active = null,
|
||||||
@@ -134,8 +135,7 @@ class PageRules implements API
|
|||||||
$options['priority'] = $priority;
|
$options['priority'] = $priority;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
$query = $this->adapter->patch('zones/' . $zoneID . '/pagerules/' . $ruleID, $options);
|
||||||
$query = $this->adapter->patch('zones/' . $zoneID . '/pagerules', $options);
|
|
||||||
|
|
||||||
$this->body = json_decode($query->getBody());
|
$this->body = json_decode($query->getBody());
|
||||||
|
|
||||||
|
|||||||
@@ -49,7 +49,7 @@ class PageRulesTest extends TestCase
|
|||||||
->method('get')
|
->method('get')
|
||||||
->with(
|
->with(
|
||||||
$this->equalTo('zones/023e105f4ecef8ad9ca31a8372d0c353/pagerules'),
|
$this->equalTo('zones/023e105f4ecef8ad9ca31a8372d0c353/pagerules'),
|
||||||
$this->equalTo([
|
$this->equalTo([
|
||||||
'status' => 'active',
|
'status' => 'active',
|
||||||
'order' => 'status',
|
'order' => 'status',
|
||||||
'direction' => 'desc',
|
'direction' => 'desc',
|
||||||
@@ -94,7 +94,7 @@ class PageRulesTest extends TestCase
|
|||||||
$mock->expects($this->once())
|
$mock->expects($this->once())
|
||||||
->method('patch')
|
->method('patch')
|
||||||
->with(
|
->with(
|
||||||
$this->equalTo('zones/023e105f4ecef8ad9ca31a8372d0c353/pagerules'),
|
$this->equalTo('zones/023e105f4ecef8ad9ca31a8372d0c353/pagerules/9a7806061c88ada191ed06f989cc3dac'),
|
||||||
$this->equalTo([
|
$this->equalTo([
|
||||||
'targets' => $target->getArray(),
|
'targets' => $target->getArray(),
|
||||||
'actions' => $action->getArray(),
|
'actions' => $action->getArray(),
|
||||||
@@ -104,7 +104,7 @@ class PageRulesTest extends TestCase
|
|||||||
);
|
);
|
||||||
|
|
||||||
$pageRules = new \Cloudflare\API\Endpoints\PageRules($mock);
|
$pageRules = new \Cloudflare\API\Endpoints\PageRules($mock);
|
||||||
$result = $pageRules->updatePageRule('023e105f4ecef8ad9ca31a8372d0c353', $target, $action, true, 1);
|
$result = $pageRules->updatePageRule('023e105f4ecef8ad9ca31a8372d0c353', '9a7806061c88ada191ed06f989cc3dac', $target, $action, true, 1);
|
||||||
|
|
||||||
$this->assertTrue($result);
|
$this->assertTrue($result);
|
||||||
$this->assertEquals('9a7806061c88ada191ed06f989cc3dac', $pageRules->getBody()->result->id);
|
$this->assertEquals('9a7806061c88ada191ed06f989cc3dac', $pageRules->getBody()->result->id);
|
||||||
|
|||||||
Reference in New Issue
Block a user