Problem/Motivation

Error encountered:
Error: Call to undefined method Drupal\Component\Utility\Bytes::toInt() in Drupal\video\Plugin\Field\FieldWidget\VideoUploadWidget->getUploadValidators() (line 253 of modules\contrib\video\src\Plugin\Field\FieldWidget\VideoUploadWidget.php).

Steps to reproduce

Chose Video upload instead embed video on Manage Form Display
Add a node (a content), then the following error is encountered:

Error: Call to undefined method Drupal\Component\Utility\Bytes::toInt() in Drupal\video\Plugin\Field\FieldWidget\VideoUploadWidget->getUploadValidators() (line 253 of modules\contrib\video\src\Plugin\Field\FieldWidget\VideoUploadWidget.php).

CommentFileSizeAuthor
#14 3353255-fix-undefined-error-14.diff1.7 KBcodebymikey
#12 3353255-fix-undefined-error-12.diff2.38 KB_renify_

Issue fork video-3353255

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:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Moudeen created an issue. See original summary.

Moudeen’s picture

Moudeen CreditAttribution: Moudeen commented

Can you create a patch for this?

replace Bytes::toInt() with Bytes::toNumber() or update the module

Thank you

Moudeen opened merge request !16

sahil.goyal made their first commit to this issue’s fork.

sahil.goyal opened merge request !17

Moudeen opened merge request !18

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs work » Reviewed & tested by the community

Confirming #2 is the correct solution, we had a similar issue in one of our modules.

RTBC for MR!17

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

a_mitch’s picture

a_mitch CreditAttribution: a_mitch commented
Status: Reviewed & tested by the community » Needs review

Updated MR 17. There were a couple other instances of toInt() in VideoUploadWidget

keshavv’s picture

keshavv CreditAttribution: keshavv as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community

Reviewed and tested MR 17, Its working perfect. Ready to merge.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented

Thanks @a_mitch! I indeed didn't check that, just reviewed what was done. Perfect! RTBC+1

_renify_’s picture

_renify_ CreditAttribution: _renify_ as a volunteer commented
FileSize
3353255-fix-undefined-error-12.diff2.38 KB

This also fix when saving node page https://www.drupal.org/node/3201242

lisotton’s picture

lisotton CreditAttribution: lisotton at European Commission and European Union Institutions, Agencies and Bodies commented

I also tested MR 17 and it is working fine.

@_renify with your patch, you are fixing things that is not related to this issue and is fixed in #3348748.

codebymikey’s picture

codebymikey CreditAttribution: codebymikey at Zodiac Media commented
FileSize
3353255-fix-undefined-error-14.diff1.7 KB

Addressing feedback from #13, removing the accessCheck(FALSE) call.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented

Let's simply use the MR, thanks. Any maintainer?

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
CreditAttribution: vmarchuk at Centarro commented

Tested MR #17 and it is working fine. RTBC+1

mmenavas’s picture

mmenavas CreditAttribution: mmenavas as a volunteer commented

Tested patch #14 and it is working fine. RTBC+1

kristian.raychev’s picture

kristian.raychev CreditAttribution: kristian.raychev at FFW commented

I also tested MR #17 and it works fine. RTBC+1

drupalfan2’s picture

drupalfan2 CreditAttribution: drupalfan2 as a volunteer commented

+1

podarok closed merge request !18

podarok closed merge request !16

podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok at ITCare, Open Y, YMCA of the North, Drupal Ukraine Community, Jet.Dev, QS Quacquarelli Symonds, YMCA of the USA for YMCA of the USA, ITCare, QS Quacquarelli Symonds, Drupal Ukraine Community, Open Y commented
Version: 3.0.4 » 3.0.5
Status: Reviewed & tested by the community » Fixed

tnx

Status: Fixed » Closed (fixed)

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

_renify_’s picture

_renify_ CreditAttribution: _renify_ as a volunteer commented