Get latest changes from original module

.github/workflows/ci.yml

@@ -4,13 +4,8 @@ on:
   push:
   pull_request:
   workflow_dispatch:
-  # Every Wednesday at 11:30am UTC
-  schedule:
-    - cron: '30 11 * * 3'
 
 jobs:
   ci:
     name: CI
-    # Only run cron on the silverstripe account
-    if: (github.event_name == 'schedule' && github.repository_owner == 'silverstripe') || (github.event_name != 'schedule')
     uses: silverstripe/gha-ci/.github/workflows/ci.yml@v1

.github/workflows/docs.yml → .github/workflows/deploy-userhelp-docs.yml

@@ -1,13 +1,14 @@
-name: Build Docs
+name: Deploy Userhelp Docs
 on:
   push:
     branches:
-      - 'master'
+      - '5'
+      - '4'
     paths:
       - 'docs/en/userguide/**'
 jobs:
-  build:
-    name: build-docs
+  deploy:
+    name: deploy-userhelp-docs
     runs-on: ubuntu-latest
     steps:
       - name: Run build hook

.github/workflows/dispatch-ci.yml

@@ -0,0 +1,16 @@
+name: Dispatch CI
+
+on:
+  # At 11:30 AM UTC, only on Wednesday and Thursday
+  schedule:
+    - cron: '30 11 * * 3,4'
+
+jobs:
+  dispatch-ci:
+    name: Dispatch CI
+    # Only run cron on the silverstripe account
+    if: (github.event_name == 'schedule' && github.repository_owner == 'silverstripe') || (github.event_name != 'schedule')
+    runs-on: ubuntu-latest
+    steps:
+      - name: Dispatch CI
+        uses: silverstripe/gha-dispatch-ci@v1

.tx/config

@@ -1,14 +1,15 @@
 [main]
 host = https://www.transifex.com
 
-[silverstripe-contentreview.master]
+[o:silverstripe:p:silverstripe-contentreview:r:master]
 file_filter = lang/<lang>.yml
 source_file = lang/en.yml
 source_lang = en
-type = YML
+type        = YML
 
-[silverstripe-contentreview.master-js]
+[o:silverstripe:p:silverstripe-contentreview:r:master-js]
 file_filter = client/lang/src/<lang>.json
 source_file = client/lang/src/en.json
 source_lang = en
-type = KEYVALUEJSON
+type        = KEYVALUEJSON
+

client/lang/en.js

@@ -1,5 +1,5 @@
-// This file was generated by silverstripe/cow from client/lang/src/en.json.
-// See https://github.com/silverstripe/cow for details
+// This file was generated by silverstripe/tx-translator from client/lang/src/en.json.
+// See https://github.com/silverstripe/silverstripe-tx-translator for details
 if (typeof(ss) === 'undefined' || typeof(ss.i18n) === 'undefined') {
   if (typeof(console) !== 'undefined') { // eslint-disable-line no-console
     console.error('Class ss.i18n not defined');  // eslint-disable-line no-console

client/lang/sk.js

@@ -0,0 +1,11 @@
+// This file was generated by silverstripe/tx-translator from client/lang/src/sk.json.
+// See https://github.com/silverstripe/silverstripe-tx-translator for details
+if (typeof(ss) === 'undefined' || typeof(ss.i18n) === 'undefined') {
+  if (typeof(console) !== 'undefined') { // eslint-disable-line no-console
+    console.error('Class ss.i18n not defined');  // eslint-disable-line no-console
+  }
+} else {
+  ss.i18n.addDictionary('sk', {
+    "ContentReview.CONTENT_DUE_FOR_REVIEW": "Obsah určený na kontrolu"
+});
+}

client/lang/src/sk.json

@@ -0,0 +1,3 @@
+{
+    "ContentReview.CONTENT_DUE_FOR_REVIEW": "Obsah určený na kontrolu"
+}

composer.json

@@ -7,7 +7,10 @@
     "keywords": [
         "silverstripe",
         "cms",
-        "workflow"
+        "workflow",
+        "content review",
+        "review",
+        "permissions"
     ],
     "authors": [
         {
@@ -22,7 +25,7 @@
     "require": {
         "php": "^7.4 || ^8.0",
         "silverstripe/vendor-plugin": "^1",
-        "silverstripe/framework": "^4.10",
+        "silverstripe/framework": "^4.11",
         "silverstripe/cms": "^4.2",
         "silverstripe/reports": "^4.2",
         "silverstripe/siteconfig": "^4.2"
@@ -49,4 +52,4 @@
     },
     "minimum-stability": "dev",
     "prefer-stable": true
-}
+}

docs/en/userguide/index.md

@@ -1,5 +1,7 @@
+---
 title: Content review
 summary: Mark pages in the CMS with a date and an owner for future reviews.
+---
 
 ## Content review
 

lang/en.yml

@@ -15,6 +15,14 @@ en:
     PAGEOWNERUSERS: Users
     REVIEWFREQUENCY: 'Review frequency'
     REVIEWFREQUENCYDESCRIPTION: 'The review date will be set to this far in the future, whenever the page is published.'
+    db_ReviewBody: 'Review body'
+    db_ReviewFrom: 'Review from'
+    db_ReviewPeriodDays: 'Review period days'
+    db_ReviewSubject: 'Review subject'
+    many_many_ContentReviewGroups: 'Content review groups'
+    many_many_ContentReviewUsers: 'Content review users'
+  SilverStripe\ContentReview\Extensions\ContentReviewOwner:
+    many_many_SiteTreeContentReview: 'Site tree content review'
   SilverStripe\ContentReview\Extensions\SiteTreeContentReview:
     ADDGROUP: 'Add groups'
     ADDUSERS: 'Add users'
@@ -34,6 +42,14 @@ en:
     REVIEWFREQUENCYDESCRIPTION: 'The review date will be set to this far in the future whenever the page is published'
     REVIEWHEADER: 'Content review'
     SETTINGSFROM: 'Options are'
+    belongs_many_many_ContentReviewGroups: 'Content review groups'
+    belongs_many_many_ContentReviewUsers: 'Content review users'
+    db_ContentReviewType: 'Content review type'
+    db_LastEditedByName: 'Last edited by name'
+    db_NextReviewDate: 'Next review date'
+    db_OwnerNames: 'Owner names'
+    db_ReviewPeriodDays: 'Review period days'
+    has_many_ReviewLogs: 'Review logs'
   SilverStripe\ContentReview\Forms\ReviewContentHandler:
     ErrorReviewPermissionDenied: 'It seems you don''t have the necessary permissions to submit a content review'
     MarkAsReviewedAction: 'Mark as reviewed'
@@ -46,6 +62,9 @@ en:
       one: 'A Content Review Log'
       other: '{count} Content Review Logs'
     SINGULARNAME: 'Content Review Log'
+    db_Note: Note
+    has_one_Reviewer: Reviewer
+    has_one_SiteTree: 'Site tree'
   SilverStripe\ContentReview\Reports\PagesDueForReviewReport:
     ONLYMYPAGES: 'Only Show pages assigned to me'
     REVIEWDATEAFTER: 'Review date after or on'

lang/sk.yml

@@ -0,0 +1,80 @@
+sk:
+  SilverStripe\ContentReview\Extensions\ContentReviewCMSExtension:
+    ErrorItemPermissionDenied: 'Zdá sa, že nemáte potrebné oprávnenia na kontrolu tohto obsahu'
+  SilverStripe\ContentReview\Extensions\ContentReviewDefaultSettings:
+    ADDGROUP: 'Pridať skupiny'
+    ADDUSERS: 'Pridať používateľov'
+    DEFAULTSETTINGSHELP: 'Tieto nastavenia sa použijú na všetky stránky, ktoré nemajú špecifický plán kontroly obsahu.'
+    EMAILFROM: 'Z e-mailovej adresy'
+    EMAILFROM_RIGHTTITLE: 'napr.: do-not-reply@site.com'
+    EMAILSUBJECT: Predmet
+    EMAILTEMPLATE: 'Šablóna e-mailu'
+    OWNERGROUPSDESCRIPTION: 'Vlastníci stránok, ktorí sú zodpovední za kontrolu'
+    OWNERUSERSDESCRIPTION: 'Vlastníci stránok, ktorí sú zodpovední za kontrolu'
+    PAGEOWNERGROUPS: Skupiny
+    PAGEOWNERUSERS: Používatelia
+    REVIEWFREQUENCY: 'Frekvencia kontroly'
+    REVIEWFREQUENCYDESCRIPTION: 'Dátum kontroly bude nastavený na tento dátum v budúcnosti, bez ohľadu na to, kedy bude stránka zverejnená.'
+    db_ReviewBody: 'Revízny orgán'
+    db_ReviewFrom: 'Kontrola od'
+    db_ReviewPeriodDays: 'Dni periódy kontroly'
+    db_ReviewSubject: 'Predmet kontroly'
+    many_many_ContentReviewGroups: 'Skupiny kontroly obsahu'
+    many_many_ContentReviewUsers: 'Používatelia kontroly obsahu'
+  SilverStripe\ContentReview\Extensions\ContentReviewOwner:
+    many_many_SiteTreeContentReview: 'Kontrola obsahu stromu stránok'
+  SilverStripe\ContentReview\Extensions\SiteTreeContentReview:
+    ADDGROUP: 'Pridať skupiny'
+    ADDUSERS: 'Pridať používateľov'
+    CONTENTOWNERS: 'Vlastníci obsahu'
+    CONTENTREVIEW: 'Obsah určený na kontrolu'
+    CUSTOM: 'Vlastné nastavenia'
+    DISABLE: 'Zakázať kontrolu obsahu'
+    INHERIT: 'Zdediť z nadradenej stránky'
+    NEXTREVIEWDATADESCRIPTION: 'Ak nechcete vyplniť kontrolu, ponechajte pole prázdne'
+    NEXTREVIEWDATE: 'Dátum ďalšej kontroly'
+    OPTIONS: Možnosti
+    OWNERGROUPSDESCRIPTION: 'Vlastníci stránok, ktorí sú zodpovední za kontrolu'
+    OWNERUSERSDESCRIPTION: 'Vlastníci stránok, ktorí sú zodpovední za kontrolu'
+    PAGEOWNERGROUPS: Skupiny
+    PAGEOWNERUSERS: Používatelia
+    REVIEWFREQUENCY: 'Frekvencia kontroly'
+    REVIEWFREQUENCYDESCRIPTION: 'Dátum kontroly bude nastavený na tento dátum v budúcnosti, bez ohľadu na to, kedy bude stránka zverejnená.'
+    REVIEWHEADER: 'Kontrola obsahu'
+    SETTINGSFROM: 'Možnosti sú'
+    belongs_many_many_ContentReviewGroups: 'Skupiny kontroly obsahu'
+    belongs_many_many_ContentReviewUsers: 'Používatelia kontroly obsahu'
+    db_ContentReviewType: 'Typ kontroly obsahu'
+    db_LastEditedByName: 'Naposledy upravené podľa mena'
+    db_NextReviewDate: 'Dátum ďalšej kontroly'
+    db_OwnerNames: 'Mená vlastníkov'
+    db_ReviewPeriodDays: 'Dni periódy kontroly'
+    has_many_ReviewLogs: 'Záznamy z kontroly'
+  SilverStripe\ContentReview\Forms\ReviewContentHandler:
+    ErrorReviewPermissionDenied: 'Zdá sa, že nemáte potrebné oprávnenia na odoslanie kontroly obsahu'
+    MarkAsReviewedAction: 'Označiť ako skontrolované'
+    NoComments: '(bez komentára)'
+    Placeholder: 'Pridať komentáre (voliteľné)'
+    Success: 'Kontrola bola úspešne pridaná'
+  SilverStripe\ContentReview\Models\ContentReviewLog:
+    PLURALNAME: 'Záznamy z kontroly obsahu'
+    PLURALS:
+      few: '{count} záznamy z kontroly obsahu'
+      many: '{count} záznamov z kontroly obsahu'
+      one: 'Záznam z kontroly obsahu'
+      other: '{count} záznamov z kontroly obsahu'
+    SINGULARNAME: 'Záznam z kontroly obsahu'
+    db_Note: Poznámka
+    has_one_Reviewer: Kontrolór
+    has_one_SiteTree: 'Strom stránok'
+  SilverStripe\ContentReview\Reports\PagesDueForReviewReport:
+    ONLYMYPAGES: 'Zobraziť iba mne priradené stránky'
+    REVIEWDATEAFTER: 'Dátum kontroly po'
+    REVIEWDATEBEFORE: 'Dátum kontroly pred'
+    SHOWVIRTUALPAGES: 'Zobraziť virtuálne stránky'
+    TITLE: 'Stránky určené na kontrolu'
+  SilverStripe\ContentReview\Reports\PagesWithoutReviewScheduleReport:
+    TITLE: 'Stránky bez naplánovanej kontroly.'
+  SilverStripe\ContentReview\Tasks\ContentReviewEmails:
+    REVIEWPAGELINK: 'Skontrolujte stránku v CMS'
+    VIEWPUBLISHEDLINK: 'Pozrite si túto stránku na webe'

src/Extensions/ContentReviewDefaultSettings.php

@@ -130,6 +130,7 @@ public function updateCMSFields(FieldList $fields)
 
         $users = Permission::get_members_by_permission([
             'CMS_ACCESS_CMSMain',
+            'CMS_ACCESS_LeftAndMain',
             'ADMIN',
         ]);
 

src/Extensions/SiteTreeContentReview.php

@@ -397,7 +397,7 @@ public function updateSettingsFields(FieldList $fields)
             $options
         );
 
-        $users = Permission::get_members_by_permission(["CMS_ACCESS_CMSMain", "ADMIN"]);
+        $users = Permission::get_members_by_permission(['CMS_ACCESS_CMSMain', 'CMS_ACCESS_LeftAndMain', 'ADMIN']);
 
         $usersMap = $users->map("ID", "Title")->toArray();
 

src/Jobs/ContentReviewNotificationJob.php

@@ -93,7 +93,7 @@ public function process()
     {
         $this->queueNextRun();
 
-        $task = new ContentReviewEmails();
+        $task = ContentReviewEmails::create();
         $task->run(new HTTPRequest("GET", "/dev/tasks/ContentReviewEmails"));
 
         $this->currentStep = 1;

src/Tasks/ContentReviewEmails.php

@@ -3,6 +3,7 @@
 namespace SilverStripe\ContentReview\Tasks;
 
 use Page;
+use RuntimeException;
 use SilverStripe\ContentReview\Compatibility\ContentReviewCompatability;
 use SilverStripe\Control\Email\Email;
 use SilverStripe\Control\HTTPRequest;
@@ -15,18 +16,29 @@
 use SilverStripe\SiteConfig\SiteConfig;
 use SilverStripe\View\ArrayData;
 use SilverStripe\View\SSViewer;
-use SilverStripe\ContentReview\Models\ContentReviewLog;
 
 /**
  * Daily task to send emails to the owners of content items when the review date rolls around.
  */
 class ContentReviewEmails extends BuildTask
 {
+    private array $invalid_emails = [];
+
     /**
      * @param HTTPRequest $request
+     * @throws RuntimeException
      */
     public function run($request)
     {
+        if (!$this->isValidEmail($senderEmail = SiteConfig::current_site_config()->ReviewFrom)) {
+            throw new RuntimeException(
+                sprintf(
+                    'Provided sender email address is invalid: "%s".',
+                    $senderEmail
+                )
+            );
+        }
+
         $compatibility = ContentReviewCompatability::start();
 
         // First grab all the pages with a custom setting
@@ -42,6 +54,16 @@ public function run($request)
         }
 
         ContentReviewCompatability::done($compatibility);
+
+        if (is_array($this->invalid_emails) && count($this->invalid_emails) > 0) {
+            $plural = count($this->invalid_emails) > 1 ? 's are' : ' is';
+            throw new RuntimeException(
+                sprintf(
+                    'Provided email' . $plural . ' invalid: "%s".',
+                    implode(', ', $this->invalid_emails)
+                )
+            );
+        }
     }
 
     /**
@@ -93,6 +115,13 @@ protected function notifyOwner($ownerID, SS_List $pages)
         // Prepare variables
         $siteConfig = SiteConfig::current_site_config();
         $owner = Member::get()->byID($ownerID);
+
+        if (!$this->isValidEmail($owner->Email)) {
+            $this->invalid_emails[] = $owner->Name . ': ' . $owner->Email;
+
+            return;
+        }
+
         $templateVariables = $this->getTemplateVariables($owner, $siteConfig, $pages);
 
         // Build email
@@ -159,4 +188,12 @@ protected function getTemplateVariables($recipient, $config, $pages)
             'ToEmail' => $recipient->Email,
         ];
     }
+
+    /**
+     * Check validity of email
+     */
+    protected function isValidEmail(?string $email): bool
+    {
+        return (bool) filter_var($email, FILTER_VALIDATE_EMAIL);
+    }
 }