Skip to content

Some minor tweaks#57

Merged
michaelcullum merged 2 commits into
phpbb-extensions:masterfrom
iMattPro:release-prep
Nov 20, 2016
Merged

Some minor tweaks#57
michaelcullum merged 2 commits into
phpbb-extensions:masterfrom
iMattPro:release-prep

Conversation

@iMattPro
Copy link
Copy Markdown
Contributor

No description provided.

@iMattPro iMattPro added this to the 1.0.2 milestone Nov 20, 2016
Comment thread ext.php
public function is_enableable()
{
return class_exists('ZipArchive') && ($this->phpbb_legacy_compatibility() || $this->phpbb_current_compatibility());
return $this->php_requirements() && ($this->phpbb_31x_compatible() || $this->phpbb_current_compatible());
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Where is the phpbb_31x_compatible method?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just view the page without the edits and you'll see it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That's odd. I uncollapsed the whole page and it didn't show up. My bad.

@michaelcullum michaelcullum merged commit caea4c6 into phpbb-extensions:master Nov 20, 2016
@iMattPro iMattPro deleted the release-prep branch November 20, 2016 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants