New issue

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

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

Already on GitHub? Sign in to your account

UserMgmtLog fatal conversion #123

Closed
dqwiki opened this Issue Feb 4, 2017 · 3 comments

Comments

Projects
None yet
1 participant
@dqwiki
Member

dqwiki commented Feb 4, 2017

Catchable fatal error: Object of class UTRSUser could not be converted to string in /usr/UTRSgit/utrs/public_html/src/userMgmtLogObject.php on line 51

$hideTarget didn't like ints in testing, so I had switched it to string, apparently it doesn't like strings either, so we need to get to the bottom of where these are being inputted and switch it to the one method.

@dqwiki dqwiki added the bug label Feb 4, 2017

@dqwiki dqwiki self-assigned this Feb 4, 2017

dqwiki added a commit that referenced this issue Feb 5, 2017

@dqwiki

This comment has been minimized.

Member

dqwiki commented Feb 5, 2017

It appears a default assignment (if not directly issued by calling function) in the insert function made this an integer.

dqwiki added a commit that referenced this issue Feb 5, 2017

dqwiki added a commit that referenced this issue Feb 5, 2017

dqwiki added a commit that referenced this issue Feb 5, 2017

dqwiki added a commit that referenced this issue Feb 5, 2017

@dqwiki dqwiki changed the title from UTRSUser $hideTarget conversion issues to UserMgmtLog fatal conversion Feb 5, 2017

@dqwiki

This comment has been minimized.

Member

dqwiki commented Feb 5, 2017

I've figured out it's not the $hideTarget var specifically but the entire query PHP is complaining about. I have a feeling it might have an issue with the null field I put in, hence why it's only complaining now.

@dqwiki

This comment has been minimized.

Member

dqwiki commented Feb 5, 2017

So what happened was a Class was returning, unlike I thought an integer was returning.
Note to self, read the error and check the class you are in.

@dqwiki dqwiki referenced this issue Feb 5, 2017

Merged

Oauth fixes push #124

dqwiki added a commit that referenced this issue Feb 5, 2017

Oauth fixes push (#124)
Oauth fixes for bugs #122 #123

@dqwiki dqwiki closed this Feb 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment