diff --git a/Admin/Install/Media/defaultReport.pdf.php b/Admin/Install/Media/defaultReport.pdf.php
index e69de29..04ce5a4 100755
--- a/Admin/Install/Media/defaultReport.pdf.php
+++ b/Admin/Install/Media/defaultReport.pdf.php
@@ -0,0 +1,13 @@
+id;
}
-
- /**
- * Get type.
- *
- * @return int
- *
- * @since 1.0.0
- */
- public function getType() : int
- {
- return $this->type;
- }
-
- /**
- * Get subtype.
- *
- * @return string
- *
- * @since 1.0.0
- */
- public function getTrigger() : string
- {
- return $this->trigger;
- }
-
- /**
- * Get Module.
- *
- * @return null|string
- *
- * @since 1.0.0
- */
- public function getModule() : ?string
- {
- return $this->module;
- }
-
- /**
- * Get reference.
- *
- * If existing this can be a reference to another model
- *
- * @return null|string
- *
- * @since 1.0.0
- */
- public function getRef() : ?string
- {
- return $this->ref;
- }
-
- /**
- * Get content.
- *
- * @return null|string
- *
- * @since 1.0.0
- */
- public function getContent() : ?string
- {
- return $this->content;
- }
-
- /**
- * Get old value.
- *
- * @return null|string
- *
- * @since 1.0.0
- */
- public function getOld() : ?string
- {
- return $this->old;
- }
-
- /**
- * Get new value.
- *
- * @return null|string
- *
- * @since 1.0.0
- */
- public function getNew() : ?string
- {
- return $this->new;
- }
-
- /**
- * Get ip.
- *
- * @return int
- *
- * @since 1.0.0
- */
- public function getIp() : int
- {
- return $this->ip;
- }
}
diff --git a/Theme/Backend/audit-list.tpl.php b/Theme/Backend/audit-list.tpl.php
index 4149bfe..1e335aa 100755
--- a/Theme/Backend/audit-list.tpl.php
+++ b/Theme/Backend/audit-list.tpl.php
@@ -108,17 +108,17 @@ echo $this->getData('nav')->render(); ?>
| = $audit->getId(); ?>
| = $this->printHtml($audit->getModule()); ?>
- | getOld() === null) : echo $this->getHtml('CREATE'); ?>
- getOld() !== null && $audit->getNew() !== null) : echo $this->getHtml('UPDATE'); ?>
- getNew() === null) : echo $this->getHtml('DELETE'); ?>
+ | old === null) : echo $this->getHtml('CREATE'); ?>
+ old !== null && $audit->new !== null) : echo $this->getHtml('UPDATE'); ?>
+ new=== null) : echo $this->getHtml('DELETE'); ?>
getHtml('UNKNOWN'); ?>
- | = $audit->getType(); ?>
- | = $audit->getTrigger(); ?>
+ | = $this->printHtml($audit->type); ?>
+ | = $this->printHtml($audit->trigger); ?>
| = $this->printHtml(
$this->renderUserName('%3$s %2$s %1$s', [$audit->createdBy->name1, $audit->createdBy->name2, $audit->createdBy->name3, $audit->createdBy->login])
); ?>
- | = $this->printHtml($audit->getRef()); ?>
+ | = $this->printHtml($audit->ref); ?>
| = $audit->createdAt->format('Y-m-d H:i:s'); ?>
diff --git a/Theme/Backend/audit-single.tpl.php b/Theme/Backend/audit-single.tpl.php
index 0977a4c..d976356 100755
--- a/Theme/Backend/audit-single.tpl.php
+++ b/Theme/Backend/audit-single.tpl.php
@@ -43,39 +43,39 @@ echo $this->getData('nav')->render();
| = $this->getHtml('Action'); ?>
- | getOld() === null) : echo $this->getHtml('CREATE'); ?>
- getOld() !== null && $audit->getNew() !== null) : echo $this->getHtml('UPDATE'); ?>
- getNew() === null) : echo $this->getHtml('DELETE'); ?>
+ | old === null) : echo $this->getHtml('CREATE'); ?>
+ old !== null && $audit->new !== null) : echo $this->getHtml('UPDATE'); ?>
+ new === null) : echo $this->getHtml('DELETE'); ?>
getHtml('UNKNOWN'); ?>
|
| = $this->getHtml('Type'); ?>
- | = $audit->getType(); ?>
+ | = $audit->type; ?>
|
| = $this->getHtml('By'); ?>
| = $audit->createdBy->name1; ?> = $audit->createdBy->name2; ?>
|
| = $this->getHtml('Ref'); ?>
- | = $this->printHtml($audit->getRef()); ?>
+ | = $this->printHtml($audit->ref); ?>
|
| = $this->getHtml('Date'); ?>
| = $this->getDateTime($audit->createdAt, 'very_long'); ?>
|
| = $this->getHtml('Module'); ?>
- | = $audit->getModule(); ?>
+ | = $audit->module; ?>
|
| = $this->getHtml('IP'); ?>
- | = \long2ip($audit->getIp()); ?>
+ | = \long2ip($audit->ip); ?>
|
= \phpOMS\Utils\StringUtils::createDiffMarkup(
- ViewAbstract::html(\stripos($audit->getOld() ?? '', '{') === 0
- ? \json_encode(\json_decode($audit->getOld()), \JSON_PRETTY_PRINT)
- : $audit->getOld() ?? ''
+ ViewAbstract::html(\stripos($audit->old ?? '', '{') === 0
+ ? \json_encode(\json_decode($audit->old), \JSON_PRETTY_PRINT)
+ : $audit->old ?? ''
),
- ViewAbstract::html(\stripos($audit->getNew() ?? '', '{') === 0
- ? \json_encode(\json_decode($audit->getNew()), \JSON_PRETTY_PRINT)
- : $audit->getNew() ?? ''
+ ViewAbstract::html(\stripos($audit->new ?? '', '{') === 0
+ ? \json_encode(\json_decode($audit->new), \JSON_PRETTY_PRINT)
+ : $audit->new ?? ''
),
"\n"
); ?>
diff --git a/tests/Bootstrap.php b/tests/Bootstrap.php
index d46fd31..b5d36a8 100755
--- a/tests/Bootstrap.php
+++ b/tests/Bootstrap.php
@@ -257,6 +257,25 @@ $CONFIG = [
'root' => '/',
'https' => false,
],
+ 'app' => [
+ 'path' => __DIR__,
+ 'default' => [
+ 'app' => 'Backend',
+ 'id' => 'backend',
+ 'lang' => 'en',
+ 'theme' => 'Backend',
+ 'org' => 1,
+ ],
+ 'domains' => [
+ '127.0.0.1' => [
+ 'app' => 'Backend',
+ 'id' => 'backend',
+ 'lang' => 'en',
+ 'theme' => 'Backend',
+ 'org' => 1,
+ ],
+ ],
+ ],
'socket' => [
'master' => [
'host' => '127.0.0.1',
diff --git a/tests/Controller/ApiControllerTest.php b/tests/Controller/ApiControllerTest.php
index da05efd..cdcc312 100755
--- a/tests/Controller/ApiControllerTest.php
+++ b/tests/Controller/ApiControllerTest.php
@@ -97,13 +97,13 @@ final class ApiControllerTest extends \PHPUnit\Framework\TestCase
foreach($logs as $log) {
if ($log->getId() > 0
- && $log->getType() === 1
- && $log->getTrigger() === 'test-trigger'
- && $log->getModule() === 'Auditor'
- && $log->getRef() === 'abc'
- && $log->getContent() === 'def'
- && $log->getOld() === null
- && $log->getNew() === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
+ && $log->type === 1
+ && $log->trigger === 'test-trigger'
+ && $log->module === 'Auditor'
+ && $log->ref === 'abc'
+ && $log->content === 'def'
+ && $log->old === null
+ && $log->new === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
) {
self::assertTrue(true);
return;
@@ -126,13 +126,13 @@ final class ApiControllerTest extends \PHPUnit\Framework\TestCase
$found = false;
foreach($logs as $log) {
if ($log->getId() > 0
- && $log->getType() === 1
- && $log->getTrigger() === 'test-trigger'
- && $log->getModule() === 'Auditor'
- && $log->getRef() === 'abc'
- && $log->getContent() === 'def'
- && $log->getOld() === \json_encode(['id' => 2, 'test' => true], \JSON_PRETTY_PRINT)
- && $log->getNew() === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
+ && $log->type === 1
+ && $log->trigger === 'test-trigger'
+ && $log->module === 'Auditor'
+ && $log->ref === 'abc'
+ && $log->content === 'def'
+ && $log->old === \json_encode(['id' => 2, 'test' => true], \JSON_PRETTY_PRINT)
+ && $log->new === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
) {
$found = true;
break;
@@ -168,13 +168,13 @@ final class ApiControllerTest extends \PHPUnit\Framework\TestCase
foreach($logs as $log) {
if ($log->getId() > 0
- && $log->getType() === 1
- && $log->getTrigger() === 'test-trigger'
- && $log->getModule() === 'Auditor'
- && $log->getRef() === 'abc'
- && $log->getContent() === 'def'
- && $log->getOld() === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
- && $log->getNew() === null
+ && $log->type === 1
+ && $log->trigger === 'test-trigger'
+ && $log->module === 'Auditor'
+ && $log->ref === 'abc'
+ && $log->content === 'def'
+ && $log->old === \json_encode(['id' => 1, 'test' => true], \JSON_PRETTY_PRINT)
+ && $log->new === null
) {
self::assertTrue(true);
return;
diff --git a/tests/Models/AuditMapperTest.php b/tests/Models/AuditMapperTest.php
index 415425c..a7b3c6f 100755
--- a/tests/Models/AuditMapperTest.php
+++ b/tests/Models/AuditMapperTest.php
@@ -14,9 +14,48 @@ declare(strict_types=1);
namespace Modules\Auditor\tests\Models;
+use Modules\Admin\Models\NullAccount;
+use Modules\Auditor\Models\Audit;
+use Modules\Auditor\Models\AuditMapper;
+
/**
* @internal
*/
final class AuditMapperTest extends \PHPUnit\Framework\TestCase
{
+ /**
+ * @covers Modules\Auditor\Models\AuditMapper
+ * @group module
+ */
+ public function testCR() : void
+ {
+ $audit = new Audit(
+ new NullAccount(1),
+ 'old',
+ 'new',
+ 1,
+ 'test-trigger',
+ 'Admin',
+ 'test-ref',
+ 'test-content',
+ 10000
+ );
+
+ $id = AuditMapper::create()->execute($audit);
+ self::assertGreaterThan(0, $audit->getId());
+ self::assertEquals($id, $audit->getId());
+
+ $auditR = AuditMapper::get()
+ ->where('id', $audit->getId())
+ ->execute();
+
+ self::assertEquals($audit->type, $auditR->type);
+ self::assertEquals($audit->trigger, $auditR->trigger);
+ self::assertEquals($audit->module, $auditR->module);
+ self::assertEquals($audit->ref, $auditR->ref);
+ self::assertEquals($audit->content, $auditR->content);
+ self::assertEquals($audit->old, $auditR->old);
+ self::assertEquals($audit->new, $auditR->new);
+ self::assertEquals($audit->ip, $auditR->ip);
+ }
}
diff --git a/tests/Models/AuditTest.php b/tests/Models/AuditTest.php
index d6e04e3..842a0ba 100755
--- a/tests/Models/AuditTest.php
+++ b/tests/Models/AuditTest.php
@@ -33,14 +33,14 @@ final class AuditTest extends \PHPUnit\Framework\TestCase
{
$audit = new Audit();
self::assertEquals(0, $audit->getId());
- self::assertEquals(0, $audit->getType());
- self::assertEquals('', $audit->getTrigger());
- self::assertNull($audit->getModule());
- self::assertNull($audit->getRef());
- self::assertNull($audit->getContent());
- self::assertNull($audit->getOld());
- self::assertNull($audit->getNew());
- self::assertEquals(0, $audit->getIp());
+ self::assertEquals(0, $audit->type);
+ self::assertEquals('', $audit->trigger);
+ self::assertNull($audit->module);
+ self::assertNull($audit->ref);
+ self::assertNull($audit->content);
+ self::assertNull($audit->old);
+ self::assertNull($audit->new);
+ self::assertEquals(0, $audit->ip);
self::assertEquals(0, $audit->createdBy->getId());
self::assertInstanceOf('\DateTimeImmutable', $audit->createdAt);
}
@@ -62,14 +62,14 @@ final class AuditTest extends \PHPUnit\Framework\TestCase
\ip2long('127.0.0.1')
);
- self::assertEquals(1, $audit->getType());
- self::assertEquals('trigger', $audit->getTrigger());
- self::assertEquals(3, $audit->getModule());
- self::assertEquals('test', $audit->getRef());
- self::assertEquals('content', $audit->getContent());
- self::assertEquals('old', $audit->getOld());
- self::assertEquals('new', $audit->getNew());
- self::assertEquals(\ip2long('127.0.0.1'), $audit->getIp());
+ self::assertEquals(1, $audit->type);
+ self::assertEquals('trigger', $audit->trigger);
+ self::assertEquals(3, $audit->module);
+ self::assertEquals('test', $audit->ref);
+ self::assertEquals('content', $audit->content);
+ self::assertEquals('old', $audit->old);
+ self::assertEquals('new', $audit->new);
+ self::assertEquals(\ip2long('127.0.0.1'), $audit->ip);
self::assertEquals(0, $audit->createdBy->getId());
}
}
|