Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Forum Bugfix: ForumDraftsTable fix DataTable expects DateTimeImmutable #7610

Merged
merged 2 commits into from
Jun 10, 2024

Conversation

fhelfer
Copy link
Contributor

@fhelfer fhelfer commented Jun 5, 2024

ForumDraftsTable currently throws following error, this PR fixes it

InvalidArgumentException thrown with message "Argument 'value': expected DateTimeImmutable, got ilDateTime"

Stacktrace:
22 InvalidArgumentException in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/ComponentHelper.php:64
21 ILIAS\UI\Implementation\Component\Table\Column\Column:checkArg in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/ComponentHelper.php:113
20 ILIAS\UI\Implementation\Component\Table\Column\Column:checkArgInstanceOf in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Column/Date.php:43
19 ILIAS\UI\Implementation\Component\Table\Column\Date:format in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/DataRow.php:99
18 ILIAS\UI\Implementation\Component\Table\DataRow:getCellContent in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:573
17 ILIAS\UI\Implementation\Component\Table\Renderer:fillCells in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:524
16 ILIAS\UI\Implementation\Component\Table\Renderer:renderDataRow in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:55
15 ILIAS\UI\Implementation\Component\Table\Renderer:render in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/DefaultRenderer.php:60
14 ILIAS\UI\Implementation\DefaultRenderer:render in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:401
13 ILIAS\UI\Implementation\Component\Table\Renderer:appendTableRows in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:236
12 ILIAS\UI\Implementation\Component\Table\Renderer:renderDataTable in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/Component/Table/Renderer.php:50
11 ILIAS\UI\Implementation\Component\Table\Renderer:render in /var/www/ilias_v2/ILIAS/components/ILIAS/UI/src/Implementation/DefaultRenderer.php:60
10 ILIAS\UI\Implementation\DefaultRenderer:render in /var/www/ilias_v2/ILIAS/components/ILIAS/Forum/classes/class.ilObjForumGUI.php:1141
9 ilObjForumGUI:addDraftButtonIfDraftsExists in /var/www/ilias_v2/ILIAS/components/ILIAS/Forum/classes/class.ilObjForumGUI.php:750
8 ilObjForumGUI:getContent in /var/www/ilias_v2/ILIAS/components/ILIAS/ILIASObject/classes/class.ilObjectGUI.php:1547
7 ilObjectGUI:getCenterColumnHTML in /var/www/ilias_v2/ILIAS/components/ILIAS/Forum/classes/class.ilObjForumGUI.php:700
6 ilObjForumGUI:showThreadsObject in /var/www/ilias_v2/ILIAS/components/ILIAS/Forum/classes/class.ilObjForumGUI.php:580
5 ilObjForumGUI:executeCommand in /var/www/ilias_v2/ILIAS/components/ILIAS/UICore/classes/class.ilCtrl.php:119
4 ilCtrl:forwardCommand in /var/www/ilias_v2/ILIAS/components/ILIAS/Repository/classes/class.ilRepositoryGUI.php:246
3 ilRepositoryGUI:show in /var/www/ilias_v2/ILIAS/components/ILIAS/Repository/classes/class.ilRepositoryGUI.php:226
2 ilRepositoryGUI:executeCommand in /var/www/ilias_v2/ILIAS/components/ILIAS/UICore/classes/class.ilCtrl.php:119
1 ilCtrl:forwardCommand in /var/www/ilias_v2/ILIAS/components/ILIAS/UICore/classes/class.ilCtrl.php:92
0 ilCtrl:callBaseClass in /var/www/ilias_v2/ILIAS/public/ilias.php:24

@mjansenDatabay mjansenDatabay added bugfix php Pull requests that update Php code labels Jun 5, 2024
@mjansenDatabay mjansenDatabay self-assigned this Jun 5, 2024
@fhelfer fhelfer requested a review from mjansenDatabay June 5, 2024 10:05
@fhelfer fhelfer force-pushed the forum/bugfix/drafttable branch from 044f42a to e80b760 Compare June 5, 2024 10:57
@fhelfer fhelfer requested a review from mjansenDatabay June 5, 2024 10:57
@fhelfer
Copy link
Contributor Author

fhelfer commented Jun 10, 2024

edit: PR also adds a check for user timeformat in drafttable

fhelfer added 2 commits June 10, 2024 15:05
Change Forum Draft Timezone to system timezone

Remove Timezone from ForumDrafTable
@fhelfer fhelfer force-pushed the forum/bugfix/drafttable branch from 10cc6ae to 3dbbff6 Compare June 10, 2024 13:07
@mjansenDatabay mjansenDatabay merged commit ccbd261 into ILIAS-eLearning:trunk Jun 10, 2024
2 checks passed
@mjansenDatabay
Copy link
Contributor

Thx Fabian

@fhelfer fhelfer deleted the forum/bugfix/drafttable branch September 11, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants