Compare commits

..

No commits in common. "master" and "v2023.10.20" have entirely different histories.

10 changed files with 51 additions and 125 deletions

View File

@ -1,11 +1,3 @@
periodical 2023.11.04
===========================================================
* Require Dotclear 2.28
* Require PHP 8.1
* Fix typo
* Fix perms
* Code review (phpstan)
periodical 2023.10.20
===========================================================
* Require Dotclear 2.28

View File

@ -1,7 +1,7 @@
# README
[![Release](https://img.shields.io/badge/release-2023.11.04-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/periodical/releases)
![Date](https://img.shields.io/badge/date-2023.11.04-c44d58.svg)
[![Release](https://img.shields.io/badge/release-2023.08.15-a2cbe9.svg)](https://git.dotclear.watch/JcDenis/periodical/releases)
![Date](https://img.shields.io/badge/date-2023.08.15-c44d58.svg)
[![Dotclear](https://img.shields.io/badge/dotclear-v2.27-137bbb.svg)](https://fr.dotclear.org/download)
[![Dotaddict](https://img.shields.io/badge/dotaddict-official-9ac123.svg)](https://plugins.dotaddict.org/dc2/details/periodical)
[![License](https://img.shields.io/badge/license-GPL--2.0-ececec.svg)](https://git.dotclear.watch/JcDenis/periodical/src/branch/master/LICENSE)

View File

@ -17,7 +17,7 @@ $this->registerModule(
'Periodical',
'Published periodically entries',
'Jean-Christian Denis and contributors',
'2023.11.04',
'2023.10.20',
[
'requires' => [['core', '2.28']],
'permissions' => 'My',

View File

@ -2,10 +2,10 @@
<modules xmlns:da="http://dotaddict.org/da/">
<module id="periodical">
<name>Periodical</name>
<version>2023.11.04</version>
<version>2023.10.20</version>
<author>Jean-Christian Denis and contributors</author>
<desc>Published periodically entries</desc>
<file>https://git.dotclear.watch/JcDenis/periodical/releases/download/v2023.11.04/plugin-periodical.zip</file>
<file>https://git.dotclear.watch/JcDenis/periodical/releases/download/v2023.10.20/plugin-periodical.zip</file>
<da:dcmin>2.28</da:dcmin>
<da:details>https://git.dotclear.watch/JcDenis/periodical/src/branch/master/README.md</da:details>
<da:support>https://git.dotclear.watch/JcDenis/periodical/issues</da:support>

View File

@ -40,11 +40,6 @@ use Exception;
*/
class BackendBehaviors
{
/**
* Periods combo.
*
* @var array<string, int> $combo_period
*/
private static array $combo_period = [];
/**
@ -96,7 +91,7 @@ class BackendBehaviors
/**
* User pref for periods columns lists.
*
* @param ArrayObject<string, mixed> $cols Columns
* @param ArrayObject $cols Columns
*/
public static function adminColumnsListsV2(ArrayObject $cols): void
{
@ -117,7 +112,7 @@ class BackendBehaviors
/**
* User pref periods filters options.
*
* @param ArrayObject<string, mixed> $sorts Sort options
* @param ArrayObject $sorts Sort options
*/
public static function adminFiltersListsV2(ArrayObject $sorts): void
{
@ -133,8 +128,8 @@ class BackendBehaviors
/**
* Add columns period to posts list header.
*
* @param MetaRecord $rs Record instance
* @param ArrayObject<string, mixed> $cols Columns
* @param MetaRecord $rs record instance
* @param ArrayObject $cols Columns
*/
public static function adminPostListHeaderV2(MetaRecord $rs, ArrayObject $cols): void
{
@ -146,8 +141,8 @@ class BackendBehaviors
/**
* Add columns period to posts list values.
*
* @param MetaRecord $rs Record instance
* @param ArrayObject<string, mixed> $cols Columns
* @param MetaRecord $rs record instance
* @param ArrayObject $cols Columns
*/
public static function adminPostListValueV2(MetaRecord $rs, ArrayObject $cols): void
{
@ -230,8 +225,8 @@ class BackendBehaviors
/**
* Posts actions callback to remove period.
*
* @param ActionsPosts $pa ActionsPosts instance
* @param ArrayObject<string, mixed> $post _POST actions
* @param ActionsPosts $pa ActionsPosts instance
* @param ArrayObject $post _POST actions
*/
public static function callbackRemove(ActionsPosts $pa, ArrayObject $post): void
{
@ -251,7 +246,7 @@ class BackendBehaviors
// Remove linked period
foreach ($posts_ids as $post_id) {
self::delPeriod((int) $post_id);
self::delPeriod($post_id);
}
Notices::addSuccessNotice(__('Posts have been removed from periodical.'));
@ -261,8 +256,8 @@ class BackendBehaviors
/**
* Posts actions callback to add period.
*
* @param ActionsPosts $pa ActionsPosts instance
* @param ArrayObject<string, mixed> $post _POST actions
* @param ActionsPosts $pa ActionsPosts instance
* @param ArrayObject $post _POST actions
*/
public static function callbackAdd(ActionsPosts $pa, ArrayObject $post): void
{
@ -277,8 +272,8 @@ class BackendBehaviors
// Save action
if (!empty($post['periodical'])) {
foreach ($posts_ids as $post_id) {
self::delPeriod((int) $post_id);
self::addPeriod((int) $post_id, (int) $post['periodical']);
self::delPeriod($post_id);
self::addPeriod($post_id, (int) $post['periodical']);
}
Notices::addSuccessNotice(__('Posts have been added to periodical.'));
@ -314,9 +309,9 @@ class BackendBehaviors
/**
* Add form to post sidebar.
*
* @param ArrayObject<string, mixed> $main_items Main items
* @param ArrayObject<string, mixed> $sidebar_items Sidebar items
* @param null|MetaRecord $post Post record or null
* @param ArrayObject $main_items Main items
* @param ArrayObject $sidebar_items Sidebar items
* @param null|MetaRecord $post Post record or null
*/
public static function adminPostFormItems(ArrayObject $main_items, ArrayObject $sidebar_items, ?MetaRecord $post): void
{
@ -370,7 +365,7 @@ class BackendBehaviors
/**
* Combo of available periods.
*
* @return array<string, int> List of period
* @return array List of period
*/
private static function comboPeriod(): array
{

View File

@ -89,7 +89,7 @@ class Frontend extends Process
$posts = App::auth()->sudo(Utils::getPosts(...), $posts_params);
if (!$posts->isEmpty()) {
$cur_post = App::blog()->openPostCursor();
$cur_post = App::blgo()->openPostCursor();
while ($posts->fetch()) {
// Publish post with right date

View File

@ -225,7 +225,7 @@ class ManagePeriod extends Process
(new Form('periodicalbhv'))->method('post')->action(App::backend()->getPageURL())->fields([
(new Para())->items([
(new Label(__('Title:')))->for('period_title'),
(new Input('period_title'))->size(65)->maxlength(255)->class('maximal')->value(Html::escapeHTML($vars->period_title)),
(new Input('period_title'))->size(65)->maxlenght(255)->class('maximal')->value(Html::escapeHTML($vars->period_title)),
]),
(new Div())->class('two-boxes')->items([
(new Div())->class('box odd')->items([

View File

@ -13,102 +13,46 @@ namespace Dotclear\Plugin\periodical;
*/
class ManageVars
{
/**
* Self instance.
*
* @var ManageVars $container
*/
/** @var ManageVars $container Self instance */
private static $container;
/**
* The post form action.
*
* @var string $action
*/
/** @var string $action The post form action */
public readonly string $action;
/**
* The post form redirection.
*
* @var string $redir
*/
/** @var string $redir The post form redirection */
public readonly string $redir;
/**
* The post form periods.
*
* @var array<int, int> $periods
*/
/** @var array $periods The post form periods */
public readonly array $periods;
/**
* The post form entries.
*
* @var array<int, int> $entries
*/
/** @var array $entries The post form entries */
public readonly array $entries;
/**
* The post form period id.
*
* @var null|int $period_id
*/
/** @var null|int $period_id The post form period id */
public readonly ?int $period_id;
/**
* The psort form period title.
*
* @var string $period_title
*/
/** @var string $period_title The psort form period title */
public readonly string $period_title;
/**
* The post form period publication number.
*
* @var int $period_pub_nb
*/
/** @var int $period_pub_nb The post form period publication number */
public readonly int $period_pub_nb;
/**
* The post form period publication interval.
*
* @var string $period_pub_int
*/
/** @var string $period_pub_int The post form period publication interval */
public readonly string $period_pub_int;
/**
* The post form period current date.
*
* @var string $period_curdt
*/
/** @var string $period_curdt The post form period current date */
public readonly string $period_curdt;
/**
* The post form period end date.
*
* @var string $period_curdt
*/
/** @var string $period_curdt The post form period end date */
public readonly string $period_enddt;
/**
* Is period ID wrong .
*
* @var bool $bad_period_id
*/
/** @var bool $bad_period_id Is period ID wrong */
public readonly bool $bad_period_id;
/**
* Is period current date wrong.
*
* @var bool $bad_period_curdt
*/
/** @var bool $bad_period_curdt Is period current date wrong */
public readonly bool $bad_period_curdt;
/**
* Is period end date wrong.
*
* @var bool $bad_period_enddt
*/
/** @var bool $bad_period_enddt Is period end date wrong */
public readonly bool $bad_period_enddt;
/**

View File

@ -19,7 +19,6 @@ class My extends MyPlugin
public static function checkCustomContext(int $context): ?bool
{
return match ($context) {
// Add usage perm to backend
self::MANAGE, self::MENU => App::task()->checkContext('BACKEND')
&& App::auth()->check(App::auth()->makePermissions([
App::auth()::PERMISSION_USAGE,
@ -33,7 +32,7 @@ class My extends MyPlugin
/**
* Periods action combo.
*
* @return array<string, string>
* @return array<string,sting>
*/
public static function periodsActionCombo(): array
{
@ -46,7 +45,7 @@ class My extends MyPlugin
/**
* Period entries action combo.
*
* @return array<string, array<string, string>>
* @return array<string,array{string,string}>
*/
public static function entriesActionsCombo(): array
{
@ -64,7 +63,7 @@ class My extends MyPlugin
/**
* Periods sortby combo.
*
* @return array<string, string>
* @return array<string,string>
*/
public static function sortbyCombo(): array
{
@ -78,7 +77,7 @@ class My extends MyPlugin
/**
* Period combo.
*
* @return array<string, string>
* @return array<string,string>
*/
public static function periodCombo(): array
{

View File

@ -30,12 +30,8 @@ use Exception;
*/
class Utils
{
/**
* File lock for update.
*
* @var null|string $lock
*/
private static ?string $lock = null;
/** @var null|string $lock File lock for update */
private static $lock = null;
/**
* Get periodical table cursor.
@ -50,9 +46,9 @@ class Utils
/**
* Get periods.
*
* @param array<string, mixed>|ArrayObject<string, mixed> $params Parameters
* @param bool $count_only Only counts results
* @param SelectStatement $ext_sql Optional SelectStatement instance
* @param array|ArrayObject $params Parameters
* @param bool $count_only Only counts results
* @param SelectStatement $ext_sql Optional SelectStatement instance
*
* @return MetaRecord A record with some more capabilities
*/
@ -238,9 +234,9 @@ class Utils
/**
* Get posts related to periods.
*
* @param array<string, mixed>|ArrayObject<string, mixed> $params Parameters
* @param bool $count_only Only counts results
* @param SelectStatement $ext_sql Optional SelectStatement instance
* @param array|ArrayObject $params Parameters
* @param bool $count_only Only counts results
* @param SelectStatement $ext_sql Optional SelectStatement instance
*
* @return MetaRecord A record with some more capabilities
*/
@ -308,7 +304,7 @@ class Utils
$sql->and('P.post_status = ' . App::blog()::POST_PENDING);
}
return App::blog()->getPosts($params, $count_only, $sql);
return App::blog()->getPosts($params, $count_only, $sql) ?? MetaRecord::newFromArray([]);
}
/**