Project

General

Profile

Actions

Bug #91369

closed

AbstractProvider->identifier string?

Added by Tomas Norre Mikkelsen over 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2020-05-12
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

I've been doing some PHPStan work with the TYPO3 Crawler, and found that I use

$crawlerConfiguration = BackendUtility::getRecordWSOL($this->table, $this->identifier); (1) Where the $this->identifier is a string as it's inherited from TYPO3\CMS\Backend\ContextMenu\ItemProviders\AbstractProvider (2) but should have been int according to BackendUtility::getRecordWSOL().

I'm not sure if I'm doing a faulty implementation or if the AbstractProvider Identifier should be different.

Hope you can help shed some light on this.

1) https://github.com/AOEpeople/crawler/blob/master/Classes/ContextMenu/ItemProvider.php#L70
2) https://github.com/TYPO3/TYPO3.CMS/blob/master/typo3/sysext/backend/Classes/ContextMenu/ItemProviders/AbstractProvider.php#L74


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #92111: Fix phpstan checkFunctionArgumentTypes errors in ext:backendClosed2020-10-17

Actions
Actions #1

Updated by Christian Kuhn over 3 years ago

  • Status changed from New to Closed

Thanks. We did lots of fixes like that meanwhile.

The getRecordWSOL() call now has an (int) cast for the identifier.

https://review.typo3.org/c/Packages/TYPO3.CMS/+/65466/14/typo3/sysext/backend/Classes/ContextMenu/ItemProviders/RecordProvider.php

Closing issue.

Actions #2

Updated by Christian Kuhn over 3 years ago

  • Related to Task #92111: Fix phpstan checkFunctionArgumentTypes errors in ext:backend added
Actions

Also available in: Atom PDF