Michael Hoennig
2022-10-20 22bca20c49c362b2735c3933f2da975da9e55808
code cleanup: remove unnecessary "raise notice"
3 files modified
10 ■■■■■ changed files
src/main/resources/db/changelog/050-rbac-base.sql 8 ●●●●● patch | view | raw | blame | history
src/main/resources/db/changelog/057-rbac-role-builder.sql 1 ●●●● patch | view | raw | blame | history
src/main/resources/db/changelog/118-test-customer-test-data.sql 1 ●●●● patch | view | raw | blame | history
src/main/resources/db/changelog/050-rbac-base.sql
@@ -259,9 +259,6 @@
    roleTypeFromRoleIdName = split_part(roleParts, '#', 3);
    objectUuidOfRole = findObjectUuidByIdName(objectTableFromRoleIdName, objectNameFromRoleIdName);
    raise notice $sql$findObjectUuidByIdName('%', '%') = %;$sql$, objectTableFromRoleIdName, objectNameFromRoleIdName, objectUuidOfRole;
    raise notice 'finding %, % (%), %', objectTableFromRoleIdName, objectNameFromRoleIdName, objectUuidOfRole, roleTypeFromRoleIdName;
    select uuid
        from RbacRole
        where objectUuid = objectUuidOfRole
@@ -406,7 +403,6 @@
    refId         uuid;
    permissionIds uuid[] = array []::uuid[];
begin
    raise notice 'createPermission for: % %', forObjectUuid, permitOps;
    if (forObjectUuid is null) then
        raise exception 'forObjectUuid must not be null';
    end if;
@@ -418,7 +414,6 @@
        loop
            refId = (select uuid from RbacPermission where objectUuid = forObjectUuid and op = permitOps[i]);
            if (refId is null) then
                raise notice 'createPermission: % %', forObjectUuid, permitOps[i];
                insert
                    into RbacReference ("type")
                    values ('RbacPermission')
@@ -427,11 +422,9 @@
                    into RbacPermission (uuid, objectUuid, op)
                    values (refId, forObjectUuid, permitOps[i]);
            end if;
            raise notice 'addPermission: %', refId;
            permissionIds = permissionIds || refId;
        end loop;
    raise notice 'createPermissions returning: %', permissionIds;
    return permissionIds;
end;
$$;
@@ -559,7 +552,6 @@
create or replace procedure grantPermissionsToRole(roleUuid uuid, permissionIds uuid[])
    language plpgsql as $$
begin
    raise notice 'grantPermissionsToRole: % -> %', roleUuid, permissionIds;
    if cardinality(permissionIds) = 0 then return; end if;
    for i in array_lower(permissionIds, 1)..array_upper(permissionIds, 1)
src/main/resources/db/changelog/057-rbac-role-builder.sql
@@ -55,7 +55,6 @@
    userUuid      uuid;
    grantedByRoleUuid uuid;
begin
    raise notice 'will createRole for %', roleDescriptor;
    roleUuid := createRole(roleDescriptor);
    if cardinality(permissions)  >0 then
src/main/resources/db/changelog/118-test-customer-test-data.sql
@@ -36,7 +36,6 @@
    custRowId = uuid_generate_v4();
    custAdminName = 'customer-admin@' || custPrefix || '.example.com';
    raise notice 'creating customer %:%', custReference, custPrefix;
    insert
        into test_customer (reference, prefix, adminUserName)
        values (custReference, custPrefix, custAdminName);