Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add suggestion for "ext-posix" #54

Merged
merged 1 commit into from
Jan 23, 2020
Merged

Add suggestion for "ext-posix" #54

merged 1 commit into from
Jan 23, 2020

Conversation

phansys
Copy link
Member

@phansys phansys commented Jan 22, 2020

userLogin() method calls posix_getuid() if the function is available:

PhpZabbixApi/build/templates/abstract.tpl.php

Line 427 in a5944bb

$uid = function_exists('posix_getuid') ? posix_getuid() : -1;

@phansys phansys added enhancement minor Changes labeled with "minor" can be published on minor releases, respecting SemVer. labels Jan 22, 2020
phansys added a commit that referenced this pull request Jan 23, 2020
@phansys
minor #54 Add suggestion for "ext-posix" (phansys)
bba8383 
This PR was merged into the master branch.

Discussion
----------

`userLogin()` method calls `posix_getuid()` if the function is available:
https://github.com/confirm/PhpZabbixApi/blob/a5944bbbcc6e69dde39732601e2542ff725d64f6/build/templates/abstract.tpl.php#L427

Commits
-------

5453695 Add suggestion for "ext-posix"
@phansys phansys merged commit 5453695 into confirm:master Jan 23, 2020
@phansys phansys deleted the dependencies branch January 23, 2020 15:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@n4huel n4huel n4huel approved these changes

Assignees
No one assigned
Labels
enhancement minor Changes labeled with "minor" can be published on minor releases, respecting SemVer.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@phansys @n4huel