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

Plomino_Readers doesn't work when username different from userid #830

Open
djay opened this issue Aug 24, 2017 · 5 comments
Open

Plomino_Readers doesn't work when username different from userid #830

djay opened this issue Aug 24, 2017 · 5 comments

Comments

@djay
Copy link
Contributor

djay commented Aug 24, 2017

All the documentation seems to say that userid is what is to be stored in Plomino_Readers, but the code in both 1.x and 2.x searches based on username.

https://github.com/plomino/Plomino/blob/1.16.8/Products/CMFPlomino/index/PlominoIndex.py#L158

Is the fix search based on userid? @ebrehault

@ebrehault
Copy link
Member

I remember @jean made some changes about that a long time ago, but we apparently missed that one.
We should use user id too.

@djay
Copy link
Contributor Author

djay commented Aug 24, 2017

@ebrehault I'm just worried about backwards incompatibility. We could make it so it searches for either but its a gludge.

@ebrehault
Copy link
Member

Yeah, I agre it can break compatibility, but still it seems the best option.

@djay
Copy link
Contributor Author

djay commented Aug 25, 2017

PR coming today

@djay
Copy link
Contributor Author

djay commented Aug 25, 2017

@ebrehault its a bit unclear what should happen with the special 'Anonymous' since it has a userid of None. Maybe it should at least have a userid that can't be used elsewhere in the system but there was already code that checked for Anonymous I think.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@djay @ebrehault and others