Skip to content

Update ini.zep#11954

Closed
dred86 wants to merge 5 commits intophalcon:2.1.xfrom
dred86:config-ini
Closed

Update ini.zep#11954
dred86 wants to merge 5 commits intophalcon:2.1.xfrom
dred86:config-ini

Conversation

@dred86
Copy link
Copy Markdown
Contributor

@dred86 dred86 commented Jul 4, 2016

i Try use for "_cast" return type "-> var" but this is the error:

Zephir\CompilerException: Returning type: bool but this type is not compatible with return-type hints declared in the method in /var/www/github/phalcon/config-ini/cphalcon/phalcon/config/adapter/ini.zep on line 141

                        return true;
        ---------------------------^

@dred86 dred86 mentioned this pull request Jul 4, 2016
Closed
Jurigag
Jurigag reviewed Jul 4, 2016
View reviewed changes
Comment thread phalcon/config/adapter/ini.zep Outdated
*
* @param mixed ini The array casted by `parse_ini_file`
*/
private function _cast(var ini) -> bool | null | double | int | string
Copy link
Copy Markdown
Contributor

@Jurigag Jurigag Jul 4, 2016

Choose a reason for hiding this comment

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

Fix indentation in this method.

@dred86 dred86 closed this Jul 13, 2016
@dred86 dred86 deleted the config-ini branch July 13, 2016 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dred86 @Jurigag