If I have two user default page rules setup:

- PageA applies to users with RoleA
- PageB applies to users with RoleB

I have a user with both RoleA and RoleB. I would like PageB to take precedence over PageA, but I don't see any way to switch around the order in which the pages are checked?

CommentFileSizeAuthor
#12 Screenshot from 2018-03-02 16-52-34.png261.36 KBnishantkumar155
#11 allow_weighting_to-2858918-11.patch5.02 KBemartoni
#7 evidence_01.png66.17 KBemartoni
#5 allow_weighting_to-2858918-5.patch5.87 KBthejacer87
#3 role_prefernce.jpg39.24 KBRkumar
#3 order_preference-2858918-3.patch5.31 KBRkumar

Comments

akalata created an issue. See original summary.

krknth’s picture

krknth commented

Sure, we will try to implement this feature. Thanks @akalata

Rkumar’s picture

Rkumar commented
StatusFileSize
new order_preference-2858918-3.patch5.31 KB
new role_prefernce.jpg39.24 KB

@akalata

Please review and apply the patch.
Hope this can solve your problem.
In any case, Please share your feedback.

Rkumar’s picture

Rkumar commented
Status: Active » Needs review
thejacer87’s picture

thejacer87 commented
Status: Needs review » Needs work
StatusFileSize
new allow_weighting_to-2858918-5.patch5.87 KB

i expanded this patch a little bit to allow more flexibility. Changed it to a "weight" field.

thejacer87’s picture

thejacer87 commented
Status: Needs work » Needs review
emartoni’s picture

emartoni commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new evidence_01.png66.17 KB

Patch #5 works for me!

Thanks!

chippyjacob’s picture

chippyjacob commented

This patch #5 works fine for me..

chippyjacob’s picture

chippyjacob commented

This patch #5 works fine for me..

mahaveer003’s picture

mahaveer003 commented
Status: Reviewed & tested by the community » Needs work

Hi @thejacer87, @emartoni,

We have added so many changes including coding standard So this patch is not getting applied.

So Please update your patch with the fresh pull of module.

Thanks for Support
Mahaveer

emartoni’s picture

emartoni commented
Status: Needs work » Needs review
StatusFileSize
new allow_weighting_to-2858918-11.patch5.02 KB

Hi all!

New patch attached.

Best,
Eduardo

nishantkumar155’s picture

nishantkumar155 commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Screenshot from 2018-03-02 16-52-34.png261.36 KB

I have tested #11 patch which is successfully applied.

  • mahaveer003 committed b51038d on 8.x-1.x 
    Issue #2858918 by emartoni, thejacer87, nishantkumar155, mahaveer003:...
mahaveer003’s picture

mahaveer003 commented
Status: Reviewed & tested by the community » Fixed

Thanks for Hard Work to close this issue.

Changes committed to 8.x-1.x.

Very soon we will be releasing this changes.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

chippyjacob’s picture

chippyjacob commented