diff --git a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/audits/AuditAction.java b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/audits/AuditAction.java index 41015e57..feee9d94 100644 --- a/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/audits/AuditAction.java +++ b/qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/audits/AuditAction.java @@ -46,6 +46,7 @@ import com.kingsrook.qqq.backend.core.model.data.QRecord; import com.kingsrook.qqq.backend.core.model.metadata.security.RecordSecurityLock; import com.kingsrook.qqq.backend.core.model.metadata.tables.QTableMetaData; import com.kingsrook.qqq.backend.core.model.session.QUser; +import com.kingsrook.qqq.backend.core.model.tables.QQQTableAccessor; import com.kingsrook.qqq.backend.core.utils.CollectionUtils; import com.kingsrook.qqq.backend.core.utils.Pair; @@ -177,7 +178,6 @@ public class AuditAction extends AbstractQActionFunction rs = new ArrayList<>(); rs.add(enrich(backendDetailEnricher, defineAuditUserTable(backendName))); - rs.add(enrich(backendDetailEnricher, defineAuditTableTable(backendName))); rs.add(enrich(backendDetailEnricher, defineAuditTable(backendName))); rs.add(enrich(backendDetailEnricher, defineAuditDetailTable(backendName))); return (rs); @@ -160,29 +154,6 @@ public class AuditsMetaDataProvider - /******************************************************************************* - ** - *******************************************************************************/ - private QTableMetaData defineAuditTableTable(String backendName) - { - return new QTableMetaData() - .withName(TABLE_NAME_AUDIT_TABLE) - .withBackendName(backendName) - .withAuditRules(new QAuditRules().withAuditLevel(AuditLevel.NONE)) - .withRecordLabelFormat("%s") - .withRecordLabelFields("label") - .withPrimaryKeyField("id") - .withUniqueKey(new UniqueKey("name")) - .withField(new QFieldMetaData("id", QFieldType.INTEGER)) - .withField(new QFieldMetaData("name", QFieldType.STRING)) - .withField(new QFieldMetaData("label", QFieldType.STRING)) - .withField(new QFieldMetaData("createDate", QFieldType.DATE_TIME)) - .withField(new QFieldMetaData("modifyDate", QFieldType.DATE_TIME)) - .withoutCapabilities(Capability.TABLE_INSERT, Capability.TABLE_UPDATE, Capability.TABLE_DELETE); - } - - - /******************************************************************************* ** *******************************************************************************/ @@ -215,10 +186,10 @@ public class AuditsMetaDataProvider .withBackendName(backendName) .withAuditRules(new QAuditRules().withAuditLevel(AuditLevel.NONE)) .withRecordLabelFormat("%s %s") - .withRecordLabelFields("auditTableId", "recordId") + .withRecordLabelFields("tableId", "recordId") .withPrimaryKeyField("id") .withField(new QFieldMetaData("id", QFieldType.INTEGER)) - .withField(new QFieldMetaData("auditTableId", QFieldType.INTEGER).withPossibleValueSourceName(TABLE_NAME_AUDIT_TABLE)) + .withField(new QFieldMetaData("tableId", QFieldType.INTEGER).withPossibleValueSourceName(QQQTable.TABLE_NAME)) .withField(new QFieldMetaData("auditUserId", QFieldType.INTEGER).withPossibleValueSourceName(TABLE_NAME_AUDIT_USER)) .withField(new QFieldMetaData("recordId", QFieldType.INTEGER)) .withField(new QFieldMetaData("message", QFieldType.STRING).withMaxLength(250).withBehavior(ValueTooLongBehavior.TRUNCATE_ELLIPSIS)) diff --git a/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/actions/audits/AuditActionTest.java b/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/actions/audits/AuditActionTest.java index 40b05461..b1b70e94 100644 --- a/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/actions/audits/AuditActionTest.java +++ b/qqq-backend-core/src/test/java/com/kingsrook/qqq/backend/core/actions/audits/AuditActionTest.java @@ -36,6 +36,7 @@ import com.kingsrook.qqq.backend.core.model.data.QRecord; import com.kingsrook.qqq.backend.core.model.metadata.QInstance; import com.kingsrook.qqq.backend.core.model.session.QSession; import com.kingsrook.qqq.backend.core.model.session.QUser; +import com.kingsrook.qqq.backend.core.model.tables.QQQTablesMetaDataProvider; import com.kingsrook.qqq.backend.core.processes.utils.GeneralProcessUtils; import com.kingsrook.qqq.backend.core.utils.TestUtils; import org.junit.jupiter.api.Test; @@ -59,6 +60,7 @@ class AuditActionTest extends BaseTest { QInstance qInstance = TestUtils.defineInstance(); new AuditsMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, null); + new QQQTablesMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, TestUtils.MEMORY_BACKEND_NAME, null); String userName = "John Doe"; QContext.init(qInstance, new QSession().withUser(new QUser().withFullName(userName))); @@ -69,7 +71,6 @@ class AuditActionTest extends BaseTest ///////////////////////////////////// // make sure things can be fetched // ///////////////////////////////////// - GeneralProcessUtils.getRecordByFieldOrElseThrow("auditTable", "name", TestUtils.TABLE_NAME_PERSON_MEMORY); GeneralProcessUtils.getRecordByFieldOrElseThrow("auditUser", "name", userName); QRecord auditRecord = GeneralProcessUtils.getRecordByFieldOrElseThrow("audit", "recordId", recordId); assertEquals("Test Audit", auditRecord.getValueString("message")); @@ -85,6 +86,7 @@ class AuditActionTest extends BaseTest { QInstance qInstance = TestUtils.defineInstance(); new AuditsMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, null); + new QQQTablesMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, TestUtils.MEMORY_BACKEND_NAME, null); String userName = "John Doe"; QContext.init(qInstance, new QSession().withUser(new QUser().withFullName(userName))); @@ -123,6 +125,7 @@ class AuditActionTest extends BaseTest { QInstance qInstance = TestUtils.defineInstance(); new AuditsMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, null); + new QQQTablesMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, TestUtils.MEMORY_BACKEND_NAME, null); String userName = "John Doe"; QContext.init(qInstance, new QSession().withUser(new QUser().withFullName(userName))); @@ -137,7 +140,6 @@ class AuditActionTest extends BaseTest ///////////////////////////////////// // make sure things can be fetched // ///////////////////////////////////// - GeneralProcessUtils.getRecordByFieldOrElseThrow("auditTable", "name", TestUtils.TABLE_NAME_PERSON_MEMORY); GeneralProcessUtils.getRecordByFieldOrElseThrow("auditUser", "name", userName); QRecord auditRecord = GeneralProcessUtils.getRecordByFieldOrElseThrow("audit", "recordId", recordId1); assertEquals("Test Audit", auditRecord.getValueString("message")); @@ -157,6 +159,7 @@ class AuditActionTest extends BaseTest { QInstance qInstance = TestUtils.defineInstance(); new AuditsMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, null); + new QQQTablesMetaDataProvider().defineAll(qInstance, TestUtils.MEMORY_BACKEND_NAME, TestUtils.MEMORY_BACKEND_NAME, null); String userName = "John Doe"; QContext.init(qInstance, new QSession().withUser(new QUser().withFullName(userName))); @@ -173,7 +176,6 @@ class AuditActionTest extends BaseTest ///////////////////////////////////// // make sure things can be fetched // ///////////////////////////////////// - GeneralProcessUtils.getRecordByFieldOrElseThrow("auditTable", "name", TestUtils.TABLE_NAME_PERSON_MEMORY); GeneralProcessUtils.getRecordByFieldOrElseThrow("auditUser", "name", userName); QRecord auditRecord = GeneralProcessUtils.getRecordByFieldOrElseThrow("audit", "recordId", recordId1); assertEquals("Test Audit", auditRecord.getValueString("message"));