New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot edit System Administrator role (AD_Role_ID=0) #3139

Closed
metas-lc opened this Issue Dec 5, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@metas-lc
Contributor

metas-lc commented Dec 5, 2017

Is this a bug or feature request?

bug

What is the current behavior?

error in Role window on WEBUI_Role column

Which are the steps to reproduce?

  1. login as sysadmin, open Role window and try to set WEBUI Rolle on Y
    => System aktualisiert Datensatz. WEBUI_Role= false(java.lang.Boolean) != DB: null -> New: true(java.lang.Boolean)

What is the expected or desired behavior?

should work

Note IT

Because this was a generic problem, check the same thing on AD_Client_ID=0, AD_Org_ID=0, AD_User_ID=0.
Regression: try to create some rows (new Role, new User etc) using Swing and webui.

@metas-lc metas-lc added the type:bug label Dec 5, 2017

@metas-lc metas-lc added this to the Backlog milestone Dec 5, 2017

@teosarca teosarca changed the title from error in Role window on WEBUI_Role column to Cannot edit System Administrator role (AD_Role_ID=0) Dec 5, 2017

teosarca added a commit that referenced this issue Dec 5, 2017

@metas-lc metas-lc self-assigned this Dec 8, 2017

@metas-lc

This comment has been minimized.

Show comment
Hide comment
@metas-lc

metas-lc Dec 8, 2017

Contributor
  1. swing (admin and sysadmin):
    set WEBUI_Role on Y/N OK
    create new roles and users OK
  2. webui
    set WEBUI_Role on Y/N NOK
    Happens only when you login with sysadmin, when you update System Administrator
    => the document is not saved:
    {saved: false, hasChanges: true, error: true,…}
    error:true
    hasChanges:true
    reason:"org.postgresql.util.PSQLException: ERROR: null value in column "name" violates not-null constraint↵ Detail: Failing row contains (1000050, 0, 0, Y, 2017-12-08 16:30:46+01, 100, 2017-12-08 16:30:46+01, null, 100, null, S__, null, 0, null, N, N, N, N, Y, Y, null, Y, N, N, O, N, N, null, 0, 0, null, N, N, N, N, N, N, N, N, Y, N, N, null, Y, Y, N, N, N, Y, N, null, null, N, N, N, null, null, Y, Y, N, N).↵ SQL: INSERT INTO AD_Role (AD_Client_ID,AD_Org_ID,AD_Role_ID,ConfirmQueryRecords,Created,CreatedBy,IsAccessAllOrgs,IsActive,IsCanExport,IsCanReport,IsChangeLog,IsManual,IsPersonalAccess,IsPersonalLock,IsShowAcct,IsUseUserOrgAccess,MaxQueryRecords,OverwritePriceLimit,PreferenceType,Updated,UpdatedBy,UserLevel,WEBUI_Role) VALUES (0,0,nextval('ad_role_seq'),0,TO_TIMESTAMP('2017-12-08 16:30:46','YYYY-MM-DD HH24:MI:SS'),100,'N','Y','Y','Y','N','N','N','N','N','N',0,'N','O',TO_TIMESTAMP('2017-12-08 16:30:46','YYYY-MM-DD HH24:MI:SS'),100,'S__','Y') RETURNING AD_Role_ID"
    saved:false

create new roles and users OK

Contributor

metas-lc commented Dec 8, 2017

  1. swing (admin and sysadmin):
    set WEBUI_Role on Y/N OK
    create new roles and users OK
  2. webui
    set WEBUI_Role on Y/N NOK
    Happens only when you login with sysadmin, when you update System Administrator
    => the document is not saved:
    {saved: false, hasChanges: true, error: true,…}
    error:true
    hasChanges:true
    reason:"org.postgresql.util.PSQLException: ERROR: null value in column "name" violates not-null constraint↵ Detail: Failing row contains (1000050, 0, 0, Y, 2017-12-08 16:30:46+01, 100, 2017-12-08 16:30:46+01, null, 100, null, S__, null, 0, null, N, N, N, N, Y, Y, null, Y, N, N, O, N, N, null, 0, 0, null, N, N, N, N, N, N, N, N, Y, N, N, null, Y, Y, N, N, N, Y, N, null, null, N, N, N, null, null, Y, Y, N, N).↵ SQL: INSERT INTO AD_Role (AD_Client_ID,AD_Org_ID,AD_Role_ID,ConfirmQueryRecords,Created,CreatedBy,IsAccessAllOrgs,IsActive,IsCanExport,IsCanReport,IsChangeLog,IsManual,IsPersonalAccess,IsPersonalLock,IsShowAcct,IsUseUserOrgAccess,MaxQueryRecords,OverwritePriceLimit,PreferenceType,Updated,UpdatedBy,UserLevel,WEBUI_Role) VALUES (0,0,nextval('ad_role_seq'),0,TO_TIMESTAMP('2017-12-08 16:30:46','YYYY-MM-DD HH24:MI:SS'),100,'N','Y','Y','Y','N','N','N','N','N','N',0,'N','O',TO_TIMESTAMP('2017-12-08 16:30:46','YYYY-MM-DD HH24:MI:SS'),100,'S__','Y') RETURNING AD_Role_ID"
    saved:false

create new roles and users OK

@teosarca

This comment has been minimized.

Show comment
Hide comment
@teosarca

teosarca Dec 10, 2017

Member

@metas-lc please also editing the system client, org zero, and System Administrator user.

Member

teosarca commented Dec 10, 2017

@metas-lc please also editing the system client, org zero, and System Administrator user.

metas-ts added a commit that referenced this issue Dec 10, 2017

Update ReleaseNotpes.md
[#727](metasfresh/metasfresh-webui-api#727)
Allow search of Subproducer
[#3079](#3079) Warehouse
picking group
[#1421](metasfresh/metasfresh-webui-frontend#1421)
Drop down lists remain displayed when using tab several times
[#3150](#3150) Always
keep Terms & Conditions equal per Compensation Group
[#714](metasfresh/metasfresh-webui-api#714)
login authenticate: provide an unique key for each JSONLoginRole
[#1404](metasfresh/metasfresh-webui-frontend#1404)
login form selects 2 items from roles dropdown
[#3174](#3174) Material
Receipt Candidates not generates when using Purchase Order from Sales
Order action
[#1406](metasfresh/metasfresh-webui-frontend#1406)
Can not enter "-" into any text field
[#721](metasfresh/metasfresh-webui-api#721)
picking terminal: HUs to pick: scan/filter by Locator
[#722](metasfresh/metasfresh-webui-api#722)
picking terminal: picking slot rows shall NOT have duplicate IDs
[#3139](#3139) Cannot
edit System Administrator role (AD_Role_ID=0)
[#1396](metasfresh/metasfresh-webui-frontend#1396)
View field editor: Patch request isn't sent after hit enter
[#3171](#3171) Add view
to keep track of unprocessed async work packages
[#3149](#3149) Automatic
Discount for Group Products
[#1414](metasfresh/metasfresh-webui-frontend#1414)
view: don't allow user to sort by a given column if layout says so
[#1425](metasfresh/metasfresh-webui-frontend#1425)
eslint: increase line max-len from 80 to 120.
[#716](metasfresh/metasfresh-webui-api#716)
picking terminal: scan picking slot by barcode
[#723](metasfresh/metasfresh-webui-api#723)
picking terminal: HUs to pick: Pick CUs process
[#3137](#3137) Clean
C_PaySelection
[#3160](#3160)
C_Printing_Queue_ReEnqueue with IsSelected doesn't work
[#713](metasfresh/metasfresh-webui-api#713)
Error creating bean with name 'scopedTarget.internalUserSessionData'
[#1383](metasfresh/metasfresh-webui-frontend#1383)
cannot completely delete a numeric field in grid view
[#3138](#3138) Import
BPartner window: cannot see data
[#2340](#2340) Relation
Type: changelog
[#3151](#3151)
ZoomTo-Performance Issues
[#3147](#3147) Errors wrt
C_BPartner SQL columns
[#2338](#2338) Relation
Type: fact_accounts
[#3110](#3110) Request
All does not show records
[#1407](metasfresh/metasfresh-webui-frontend#1407)
process result: open included view: handle profileId property
[#3105](#3105) Improve
Product import process
[#711](metasfresh/metasfresh-webui-api#711)
picking terminal: additional shall support grouping lines by order
[#3058](#3058)
TableRecordId relation types to support Prefix_AD_Table_ID and
Prefix_Record_ID
[#3126](#3126) Solve
issues around standalone report / jasper service
[#1393](metasfresh/metasfresh-webui-frontend#1393)
View attributes: consider row's supportAttributes property before
querying for attributes
[#1361](metasfresh/metasfresh-webui-frontend#1361)
View field editor: make sure the field value is sent to backend before
the view is deleted
@metas-lc

This comment has been minimized.

Show comment
Hide comment
@metas-lc

metas-lc Dec 11, 2017

Contributor

IT
retested the cases above => works fine, no errors

Contributor

metas-lc commented Dec 11, 2017

IT
retested the cases above => works fine, no errors

@metas-lc metas-lc closed this Dec 11, 2017

metas-ts added a commit that referenced this issue Dec 12, 2017

metas-ts added a commit that referenced this issue Dec 12, 2017

@metas-ts

This comment has been minimized.

Show comment
Hide comment
@metas-ts

metas-ts Dec 12, 2017

Member

CPed to release

Member

metas-ts commented Dec 12, 2017

CPed to release

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