Skip to content

Fixed several coding standard issues#77

Merged
ole1986 merged 3 commits intomasterfrom
cs-fixes
Jul 25, 2017
Merged

Fixed several coding standard issues#77
ole1986 merged 3 commits intomasterfrom
cs-fixes

Conversation

@lsolesen
Copy link
Copy Markdown
Collaborator

No description provided.

@lsolesen lsolesen requested a review from ole1986 July 24, 2017 17:56
Copy link
Copy Markdown
Collaborator

@ole1986 ole1986 left a comment

Choose a reason for hiding this comment

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

I would prefer to have the case instruction without the brackets "{" and "}" in line 112 - 259 in Cezpdf.php

Example:

switch (strtoupper($paper)) {
    case '4A0': 
        $size = array(0, 0, 4767.87, 6740.79); 
        break;
}

@ole1986
Copy link
Copy Markdown
Collaborator

ole1986 commented Jul 25, 2017

Travis seem to have an issue receiving PHP7 from AWS.
force merge it into master

@ole1986 ole1986 closed this Jul 25, 2017
@ole1986 ole1986 reopened this Jul 25, 2017
@ole1986 ole1986 merged commit 5626233 into master Jul 25, 2017
@ole1986 ole1986 deleted the cs-fixes branch July 25, 2017 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants