Skip to content

fix: 修复付款单权限 - #292

Open
slarkerino wants to merge 2 commits into
EasyAbp:devfrom
yuansicloud:fix-permission
Open

fix: 修复付款单权限#292
slarkerino wants to merge 2 commits into
EasyAbp:devfrom
yuansicloud:fix-permission

Conversation

@slarkerino

Copy link
Copy Markdown

修复付款单查看权限问题。不然用户查看自己的付款单也需要管理权限

@slarkerino

Copy link
Copy Markdown
Author

@gdlcf88

@gdlcf88

gdlcf88 commented Apr 16, 2025

Copy link
Copy Markdown
Member

Can you create a unit testing case to reproduce your problem?

@slarkerino

Copy link
Copy Markdown
Author

Can you create a unit testing case to reproduce your problem?

image
this is the code in the PaymentAppService, you can see it calls base.GetAsync(id) and base.GetListAsync(input). In abp ReadOnlyAppService by default it already checks GetPolicyName and GetListPolicyName in the base class. So if it is set to manage, it doesnt matter the userId

@slarkerino

slarkerino commented Apr 17, 2025

Copy link
Copy Markdown
Author

Now this should be correct.I removed manage permission from GetPolicyName, GetListPolicyName and check manage permission if userId not match. @gdlcf88

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants