Skip to content
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.

Sign up for GitHub

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

Jump to bottom

return username and password after calling request() #344

Closed
wants to merge 1 commit into from
Closed

return username and password after calling request() #344

wants to merge 1 commit into from

Conversation

alexbruy
Copy link
Contributor

No description provided.

@brushtyler
Copy link
Contributor

username and password should be both In and Out parameter I suppose, as you can set the username to be displayed and ask for password only.

@alexbruy
Copy link
Contributor Author

Maybe you are right. Unfortunately my SIP knowledge is not so good, I've tried to make them In and Out and now I get crash when reading output data. Reading SIP docs now...

@brushtyler
Copy link
Contributor

I dunno how it works and AFAICT I've never seen it in a pyQGIS sip files, so probably in similar cases a different approach was used writing sip bindings.

@ghost ghost assigned jef-n Dec 3, 2012
@alexbruy
Copy link
Contributor Author

I implemented username and password as in/out parameters in dcf3ede. Seems works fine.

@alexbruy alexbruy closed this Dec 11, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jef-n jef-n

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alexbruy @brushtyler @jef-n