Off-the-record option is now available in Story, Group, and Article

Closes #397
mikron-ia · May 30, 2024 · 61122c6 · 61122c6
1 parent 39c000d
commit 61122c6
Show file tree

Hide file tree

Showing 8 changed files with 55 additions and 9 deletions.
diff --git a/backend/views/article/_form.php b/backend/views/article/_form.php
@@ -35,6 +35,12 @@
         <?= $form->field($model, 'text_raw')->textarea(['rows' => 12]) ?>
     </div>
 
+    <?php if (!$model->isNewRecord): ?>
+        <div class="col-md-12">
+            <?= $form->field($model, 'is_off_the_record_change')->checkbox() ?>
+        </div>
+    <?php endif; ?>
+
     <div class="form-group">
         <?= Html::submitButton(
             $model->isNewRecord ? Yii::t('app', 'BUTTON_CREATE') : Yii::t('app', 'BUTTON_UPDATE'),

diff --git a/backend/views/character/_form.php b/backend/views/character/_form.php
@@ -51,7 +51,7 @@
         </div>
 
         <div class="col-md-12">
-            <?= $form->field($model, 'is_soft_change')->checkbox() ?>
+            <?= $form->field($model, 'is_off_the_record_change')->checkbox() ?>
         </div>
     <?php endif; ?>
 

diff --git a/backend/views/group/_form.php b/backend/views/group/_form.php
@@ -53,6 +53,12 @@
         ) ?>
     </div>
 
+    <?php if (!$model->isNewRecord): ?>
+        <div class="col-md-12">
+            <?= $form->field($model, 'is_off_the_record_change')->checkbox() ?>
+        </div>
+    <?php endif; ?>
+
     <div class="clearfix"></div>
 
     <div class="form-group">

diff --git a/backend/views/story/_form.php b/backend/views/story/_form.php
@@ -35,6 +35,12 @@
         <?= $form->field($model, 'long')->textarea(['rows' => 20]); ?>
     </div>
 
+    <?php if (!$model->isNewRecord): ?>
+        <div class="col-md-12">
+            <?= $form->field($model, 'is_off_the_record_change')->checkbox() ?>
+        </div>
+    <?php endif; ?>
+
     <div class="form-group col-md-2">
         <?php
         echo Html::submitButton(

diff --git a/common/models/Article.php b/common/models/Article.php
@@ -40,6 +40,8 @@ class Article extends ActiveRecord implements HasEpicControl, HasVisibility, Has
     use ToolsForEntity;
     use ToolsForDescription;
 
+    public bool $is_off_the_record_change = false;
+
     public static function tableName()
     {
         return 'article';
@@ -53,6 +55,7 @@ public function rules()
             [['text_raw'], 'string'],
             [['title', 'subtitle'], 'string', 'max' => 120],
             [['visibility'], 'string', 'max' => 20],
+            [['is_off_the_record_change'], 'boolean'],
             [
                 ['description_pack_id'],
                 'exist',
@@ -90,6 +93,7 @@ public function attributeLabels()
             'text_raw' => Yii::t('app', 'ARTICLE_TEXT'),
             'text_ready' => Yii::t('app', 'ARTICLE_TEXT'),
             'utility_bag_id' => Yii::t('app', 'UTILITY_BAG'),
+            'is_off_the_record_change' => Yii::t('app', 'CHECK_OFF_THE_RECORD_CHANGE'),
         ];
     }
 
@@ -138,9 +142,11 @@ public function beforeSave($insert)
         return parent::beforeSave($insert);
     }
 
-    public function afterSave($insert, $changedAttributes)
+    public function afterSave($insert, $changedAttributes): void
     {
-        $this->seenPack->updateRecord();
+        if (!$this->is_off_the_record_change) {
+            $this->seenPack->updateRecord();
+        }
         parent::afterSave($insert, $changedAttributes);
     }
 

diff --git a/common/models/Group.php b/common/models/Group.php
@@ -62,6 +62,8 @@ class Group extends ActiveRecord implements Displayable, HasDescriptions, HasEpi
     use ToolsForEntity;
     use ToolsForHasDescriptions;
 
+    public bool $is_off_the_record_change = false;
+
     public static function tableName(): string
     {
         return 'group';
@@ -87,6 +89,7 @@ public function rules(): array
             ],
             [['name'], 'string', 'max' => 120],
             [['visibility', 'importance_category'], 'string', 'max' => 20],
+            [['is_off_the_record_change'], 'boolean'],
             [
                 ['epic_id'],
                 'exist',
@@ -164,13 +167,16 @@ public function attributeLabels(): array
             'master_group_id' => Yii::t('app', 'GROUP_MASTER_GROUP'),
             'scribble_pack_id' => Yii::t('app', 'SCRIBBLE_PACK'),
             'utility_bag_id' => Yii::t('app', 'UTILITY_BAG'),
+            'is_off_the_record_change' => Yii::t('app', 'CHECK_OFF_THE_RECORD_CHANGE'),
         ];
     }
 
-    public function afterSave($insert, $changedAttributes)
+    public function afterSave($insert, $changedAttributes): void
     {
-        $this->seenPack->updateRecord();
-        $this->utilityBag->flagAsChanged();
+        if (!$this->is_off_the_record_change) {
+            $this->seenPack->updateRecord();
+            $this->utilityBag->flagAsChanged();
+        }
         $this->utilityBag->flagForImportanceRecalculation();
         parent::afterSave($insert, $changedAttributes);
     }

diff --git a/common/models/Story.php b/common/models/Story.php
@@ -55,6 +55,8 @@ class Story extends ActiveRecord implements Displayable, HasParameters, HasEpicC
      */
     private $parametersFormatted;
 
+    public bool $is_off_the_record_change = false;
+
     public static function tableName()
     {
         return 'story';
@@ -69,6 +71,7 @@ public function rules()
             [['key'], 'string', 'max' => 80],
             [['name'], 'string', 'max' => 120],
             [['visibility'], 'string', 'max' => 20],
+            [['is_off_the_record_change'], 'boolean'],
             [
                 ['epic_id'],
                 'exist',
@@ -100,6 +103,7 @@ public function attributeLabels()
             'data' => Yii::t('app', 'STORY_DATA'),
             'parameter_pack_id' => Yii::t('app', 'PARAMETER_PACK'),
             'utility_bag_id' => Yii::t('app', 'UTILITY_BAG'),
+            'is_off_the_record_change' => Yii::t('app', 'CHECK_OFF_THE_RECORD_CHANGE'),
         ];
     }
 
@@ -111,10 +115,12 @@ public function afterFind()
         parent::afterFind();
     }
 
-    public function afterSave($insert, $changedAttributes)
+    public function afterSave($insert, $changedAttributes): void
     {
-        $this->seenPack->updateRecord();
-        $this->utilityBag->flagAsChanged();
+        if (!$this->is_off_the_record_change) {
+            $this->seenPack->updateRecord();
+            $this->utilityBag->flagAsChanged();
+        }
         parent::afterSave($insert, $changedAttributes);
     }
 

diff --git a/console/controllers/CrutchController.php b/console/controllers/CrutchController.php
@@ -49,6 +49,7 @@ public function actionSaveCharacterSheets(): void
         $objects = CharacterSheet::find()->all();
 
         foreach ($objects as $object) {
+            /** @var CharacterSheet $object */
             $object->save(false);
         }
     }
@@ -63,6 +64,8 @@ public function actionSaveGroups(): void
         $objects = Group::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Group $object */
+            $object->is_off_the_record_change = true;
             $object->save(false);
         }
     }
@@ -77,6 +80,7 @@ public function actionSaveEpics(): void
         $objects = Epic::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Epic $object */
             $object->save(false);
         }
     }
@@ -90,6 +94,7 @@ public function actionSaveRecaps(): void
         $objects = Recap::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Recap $object */
             $object->save(false);
         }
     }
@@ -104,6 +109,8 @@ public function actionSaveStories(): void
         $objects = Story::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Story $object */
+            $object->is_off_the_record_change = true;
             $object->save(false);
         }
     }
@@ -117,6 +124,8 @@ public function actionSaveArticles(): void
         $objects = Article::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Article $object */
+            $object->is_off_the_record_change = true;
             $object->save(false);
         }
     }
@@ -131,6 +140,7 @@ public function actionSaveGames(): void
         $objects = Game::find()->all();
 
         foreach ($objects as $object) {
+            /** @var Game $object */
             $object->save(false);
         }
     }