Rio de Imagem Lda
Rank: #13553
30Views
21Comments
Solved
How to filter an agregate by the last user that logged in?
Question
Users
Beginner
Best Practices
Data
Traditional Web
Application Type
Traditional Web

Hi! So I am building an app and, based on one attribute of the User, I want to change the display of the menu (Set some things invisible). So, I set on display this:

IdentifierToInteger(GetUsers.List.Current.User.Attribute1) = 1

It is always showing me one user, no matter which user I login with. So,should I use a filter, which one?

0
0
on 26th Jan
Copy link to comment
Monika Wöhrl
Rank: #4257
Solution

Replying to Gonçalo Nogueira's comment on 27 Jan 2021 12:56:28

Hello.

Well actually this IS a problem. As I have already seen your oml file from your previous question I can tell you this:
Usually you would filter an User aggregate for the currently logged in user - as everybody here said - with the built in function GetUserId(). But this is only working with the User entity from the System.
In your application, you built your own User and Role entities. Your User-id is of type Long Integer and not a foreign key to the System User. Which is why your aggregate will still return all the users that are in the database and the container visibility is still not behaving as desired.
If you use the built in User and Roles entities, the suggestions from everyone here will work. If you want to use your own entities, I don´ t know how to retrieve the current user in a secure way.
 

See solution in context
2
0
on 27th Jan
Copy link to comment
Nick Vandebriel
Rank: #2442
Solution

Also, based on your question. You should use roles to check if a user is allowed to see something like @Rahul Sahu recommended. Don't put this in the display property of the container though.. This will just add a display: none in the inspector and can easily be tampered with to be visible. Use an if widget instead. Also I have the feeling that you don't understand the platform too well. Did you follow the training courses? I would recommend to check them out.

You can find the guided paths on the following link:

https://www.outsystems.com/training/paths/


See solution in context
2
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27
Solution

Yes that's the problem just replace the GetUser (entity action) with the builtin GetUserId() action. 

See solution in context
0
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

What do you query in the GetUsers.List.Current.User.Attribute1 query/aggregate? is it filtered by Current userId with the GetUserId() builtin action?


1
0
on 26th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553

I did not understand you but my aggregate only has the user info, nothing more. No filters, no nothing

0
0
on 26th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

Replying to Gonçalo Nogueira's comment on 26 Jan 2021 11:09:48

That explains it; you have to add the filter User.Id = GetUserId(); this will filter the aggregate on current user.

0
0
on 26th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553

Replying to Matthias Preuter's comment on 26 Jan 2021 11:12:42

But When I try to use GetUserId() i need to put something between the (), what do i put?

Menu2png.PNG

0
0
on 27th Jan
Copy link to comment
Lennart Kraak
Champion
Rank: #377

Replying to Gonçalo Nogueira's comment on 27 Jan 2021 10:35:10

Hi Gonçalo,

Matthias was refering to the Built-in action GetUserId(). This is a function that returns the Id of the current logged- in user.
Your screenshot shows GetUser(), which is an entity action to return a record of the User entity.

Regards,
Lennart

0
0
on 27th Jan
Copy link to comment
Tom Zhao
Rank: #287

Hi Gonçalo 

Matthias is right, You have to use the GetUserId action to get login user id. And set it to filter.

I don't know what the Attribute1 you mean, But show/hide the menu item mostly depends on the login user role.

If so,  insdead of using aggregate, You should use CheckRoleAction for each role.

Kind Regard

0
0
on 26th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

Hi @Gonçalo Nogueira, Could you show us the aggragate youre using, then we can point where to put the filter on GetUserId().

0
0
on 26th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553

Here you go

Menu2png.PNG

0
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27
Solution

Yes that's the problem just replace the GetUser (entity action) with the builtin GetUserId() action. 

0
0
on 27th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553
Menu3.PNG

0
0
on 27th Jan
Copy link to comment
Nick Vandebriel
Rank: #2442

Replying to Gonçalo Nogueira's comment on 27 Jan 2021 12:39:34

Hi Goncalo, 

Not really sure what you mean with "what do I put inside that", the getUserId() doesn't need anything between the brackets. It will just return the userid of the currently logged in user.

Below is a screenshot of how it should look.

 

0
0
on 27th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553

Replying to Nick Vandebriel's comment on 27 Jan 2021 12:44:09

Okay, my msitake
Anyways. I have done that and it still does not work


I have 2 accounts, one with Attribute1 = 1 and another with Attribute1 = 2
No matter with wich one I login. It does not show the menu

Menu4.PNG

0
0
on 27th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553
Menu5.PNG

0
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

Replying to Gonçalo Nogueira's comment on 27 Jan 2021 12:50:12

It's in the wrong order User.Id = GetUserId(); but that is not the problem i think

What does the worning say?

0
0
on 27th Jan
Copy link to comment
Rio de Imagem Lda
Rank: #13553

Replying to Matthias Preuter's comment on 27 Jan 2021 12:53:29

Warning says that I am comparing two diferent types of data, that I should use a data conversion function or recheck the expression
But I dont think thats the problem because no matter whcih account I login, I tdoes not show me nothing

0
0
on 27th Jan
Copy link to comment
Monika Wöhrl
Rank: #4257
Solution

Replying to Gonçalo Nogueira's comment on 27 Jan 2021 12:56:28

Hello.

Well actually this IS a problem. As I have already seen your oml file from your previous question I can tell you this:
Usually you would filter an User aggregate for the currently logged in user - as everybody here said - with the built in function GetUserId(). But this is only working with the User entity from the System.
In your application, you built your own User and Role entities. Your User-id is of type Long Integer and not a foreign key to the System User. Which is why your aggregate will still return all the users that are in the database and the container visibility is still not behaving as desired.
If you use the built in User and Roles entities, the suggestions from everyone here will work. If you want to use your own entities, I don´ t know how to retrieve the current user in a secure way.
 

2
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

Replying to Monika Wöhrl's comment on 27 Jan 2021 13:37:22

That explains a lot; didn't see that coming.... Please use standards @Gonçalo Nogueira, this is really not the way to go. Take the advice of @Nick Vandebriel  and follow the guided path online training.

0
0
on 27th Jan
Copy link to comment
Nick Vandebriel
Rank: #2442

Hi Goncalo try putting getuserid() instead of getuser()

0
0
on 27th Jan
Copy link to comment
Rahul Sahu
Rank: #88

HI, 

i think you should try roles for it. 

create a role and based on a role you can hide and show menu.


Regards
RS

2
0
on 27th Jan
Copy link to comment
Matthias Preuter
mvp_badge
MVP
Rank: #27

Agree, but this query isn't working because the wrong action is used.

1
0
on 27th Jan
Copy link to comment
Nick Vandebriel
Rank: #2442
Solution

Also, based on your question. You should use roles to check if a user is allowed to see something like @Rahul Sahu recommended. Don't put this in the display property of the container though.. This will just add a display: none in the inspector and can easily be tampered with to be visible. Use an if widget instead. Also I have the feeling that you don't understand the platform too well. Did you follow the training courses? I would recommend to check them out.

You can find the guided paths on the following link:

https://www.outsystems.com/training/paths/


2
0
on 27th Jan
Copy link to comment