Problem/Motivation

Let's get ready for d11 !!

API changes

Before:

'#upload_validators' => array(
  'file_validate_extensions' => array('jpg jpeg gif png txt doc xls pdf ppt
   pps odt ods odp'),
);

After:

'#upload_validators' => [
  'FileExtension' => ['extensions' => 'jpg jpeg gif png txt doc xls pdf ppt
   pps odt ods odp'],
];
CommentFileSizeAuthor
#5 Screenshot 2024-12-06 at 10.31.12 AM.png114.66 KBrahul17

Issue fork plupload-3456524

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

budalokko created an issue. See original summary.

znerol’s picture

Status: Needs work » Reviewed & tested by the community

Verified this while working on the Drupal 11 port of Feeds Plupload Fetcher. Did read through the changes and I cannot find any place where something is missing.

Also verified that the file extension validation works using manual testing.

rahul17 made their first commit to this issue’s fork.

rahul17’s picture

Status: Reviewed & tested by the community » Needs review
Issue tags: +Drupal 11 compatibility
StatusFileSize
new114.66 KB

Fixed upgrade status issues.

Class Drupal\plupload\Element\PlUploadFile extends deprecated class Drupal\Core\Render\Element\FormElement. Deprecated in drupal:10.3.0 and is removed from drupal:12.0.0. use Drupal\Core\Render\Element\FormElementBase instead.

Functionality is working fine in Drupal 10 and Drupal 11.
Kindly verify.

znerol’s picture

Nice catch! In that case core_version_requirement should be set to ^10.3 || ^11 I guess?

znerol’s picture

Status: Needs review » Reviewed & tested by the community

  • budalokko committed dd7273b6 on 2.2.x
    Issue #3456524 by budalokko, rahul17, znerol: Drupal 11 compatibility
    
budalokko’s picture

Thank you all for the work !! New release available -> https://www.drupal.org/project/plupload/releases/2.2.0

budalokko’s picture

Status: Reviewed & tested by the community » Fixed
znerol’s picture

Great! Thanks @budalokko

bwaindwain’s picture

Issue summary: View changes

Fixed summary before & after according to https://www.drupal.org/node/3363700

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.