You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Gautam Borad <gb...@gmail.com> on 2016/02/16 12:54:13 UTC

Review Request 43479: RANGER-799 : Ranger UI fixes - partial search not working on Policy listing page

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43479/
-----------------------------------------------------------

Review request for ranger, Alok Lal, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Ramesh Mani, Selvamohan Neethiraj, and Velmurugan Periasamy.


Bugs: RANGER-799
    https://issues.apache.org/jira/browse/RANGER-799


Repository: ranger


Description
-------

Patch contains solutions of List of UI issues : 
1. (Policy) Partial search not working on policy listing page.
2. (Audit>Admin) Audit of Update Service shows wrong values. 
3. Corrected error message if Lastname is not entered.


Diffs
-----

  security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java 4a14ab9 
  security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java 7cbf599 
  security-admin/src/main/java/org/apache/ranger/util/RangerRestUtil.java eb34b42 
  security-admin/src/main/webapp/scripts/controllers/Controller.js 1ac58cf 
  security-admin/src/main/webapp/scripts/mgrs/SessionMgr.js 6449c50 
  security-admin/src/main/webapp/scripts/models/VXPortalUser.js f9b7055 
  security-admin/src/main/webapp/scripts/modules/XALinks.js 8efd01d 
  security-admin/src/main/webapp/scripts/utils/XAUtils.js aa904a8 
  security-admin/src/main/webapp/scripts/views/common/BreadCrumbs.js 136d9a0 
  security-admin/src/main/webapp/scripts/views/permissions/ModulePermissionForm.js e311f1e 
  security-admin/src/main/webapp/scripts/views/permissions/ModulePermsTableLayout.js 354992a 
  security-admin/src/main/webapp/scripts/views/policies/PermissionList.js ff72ced 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyCreate.js a4454bd 
  security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js 59aac0d 
  security-admin/src/main/webapp/scripts/views/reports/OperationDiffDetail.js 5f6d156 
  security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js c9a0847 
  security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js 63b5d35 
  security-admin/src/main/webapp/scripts/views/service/ServiceCreate.js 6f7d5b9 
  security-admin/src/main/webapp/scripts/views/user/UserProfile.js ca0c893 
  security-admin/src/main/webapp/scripts/views/users/UserTableLayout.js d9d6e8a 
  security-admin/src/main/webapp/templates/reports/AssetOperationDiff_tmpl.html 7f0cef6 
  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyDeleteDiff_tmpl.html bf37324 
  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyDiff_tmpl.html 353baa9 
  security-admin/src/main/webapp/templates/reports/PlugableServicePolicyUpdateDiff_tmpl.html 727c998 

Diff: https://reviews.apache.org/r/43479/diff/


Testing
-------

Verified : 

1. Partial search functionality on policy listing page : 
   Consider, we have 3 policies
    test_hdfs-1-20151214074249
    test1
    test_policy
Now if we search with "policy name:test" it shows all three policies.

2. Logs of Update service on Audit -> Admin tab shows correct mapping of old values to new values.
3. Verified changed validation message for Lastname.


Thanks,

Gautam Borad