Fatal error when saving new topic

Created on 9 November 2023, 8 months ago

Updated 29 March 2024, 3 months ago

Problem/Motivation

A fatal error is triggered when creating new content without manually set creation date.

Steps to reproduce

Go to one of the groups.
Create new topic
Do not fill in the creation date
Save the new topic

Topic is not saved due to fatal error:

Error: Call to a member function getTimestamp() on null in _social_core_created_date_validation() (line 1095 of profiles/contrib/open_social/modules/social_features/social_core/social_core.module).

Proposed resolution

The code within the _social_core_created_date_validation() function does check if the form has any value but not of an actual date is set. This check should be added.

function _social_core_created_date_validation(array $form, FormStateInterface $form_state): void {
  $created = $form_state->getValue('created');
  if (!$created) {
    return;
  }

  $date = end($created);
  $date = $date['value'];
  if (!$date instanceof DrupalDateTime) {
    return;
  }

  if ($date->getTimestamp() > time()) {
    $form_state->setErrorByName('created', t('The created date cannot be in the future.'));
  }
}

Remaining tasks

Update code.

🐛 Bug report

Status

Needs work

Version

12.0

Component

Code (back-end)

Created by

🇧🇪Belgium zero2one

Live updates comments and jobs are added and updated live.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Comments & Activities

Issue created by @zero2one
Comment 8 months ago →
🇧🇪Belgium zero2one
Comment 8 months ago →
🇧🇪Belgium zero2one
Patch to fix the issue.

See also: https://github.com/goalgorilla/open_social/pull/3602
Status changed to Needs work 6 months ago12:14pm 3 January 2024
Comment 6 months ago →
🇳🇱Netherlands Kingdutch
A test needs to be provided to demonstrate the bug and prevent regression, especially since the expectation is that this is covered by our existing test coverage and Drupal should provide a created date for topics out of the box.
Comment 3 months ago →
🇫🇷France ohmdesbois
Hello,
I encounter the same problem after updating to version 12.3.
Opensocial content does not have a creation date by default.
Previously, the date of creation of event, topic, page, etc, was entered on today's date by default.
Custom content does not have this problem.
Has anything changed about this?
THANKS

contrib.social Blog FAQ Discussions

Production build 0.69.0 2024