Skip to content
This repository has been archived by the owner on Nov 2, 2022. It is now read-only.

Add automatic_https_rewrites() and change_automatic_https_rewrites() #69

Merged
merged 4 commits into from
Aug 2, 2017

Conversation

shadowlmd
Copy link

@shadowlmd shadowlmd commented Jul 14, 2017

For #70.

*
* @param string $zone_identifier API item identifier tag
*/
public function automatic_https_rewrites($zone_identifier)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Method name Settings::automatic_https_rewrites is not in camel caps format

* @param string $zone_identifier API item identifier tag
* @param string|null $value Value of the zone setting (default: off)
*/
public function change_automatic_https_rewrites($zone_identifier, $value = null)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Method name Settings::change_automatic_https_rewrites is not in camel caps format

*
* @param string $zone_identifier API item identifier tag
*/
public function automatic_https_rewrites($zone_identifier)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Method name "Settings::automatic_https_rewrites" is not in camel caps format

* @param string $zone_identifier API item identifier tag
* @param string|null $value Value of the zone setting (default: off)
*/
public function change_automatic_https_rewrites($zone_identifier, $value = null)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Method name "Settings::change_automatic_https_rewrites" is not in camel caps format

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 3.073% when pulling e4825e9 on shadowlmd:master into 103d4ff on jamesryanbell:master.

@shadowlmd shadowlmd changed the title automatic_https_rewrites method Add automatic_https_rewrites() and change_automatic_https_rewrites() Jul 14, 2017
@jamesryanbell jamesryanbell merged commit 68fee2a into jamesryanbell:master Aug 2, 2017
fmp777 pushed a commit to fmp777/cloudflare that referenced this pull request Dec 4, 2019
Add automatic_https_rewrites() and change_automatic_https_rewrites()
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants