1
0
mirror of synced 2025-03-06 21:06:16 +03:00

Merge pull request #6216 from malukenho/enhancement/remove-unused-param

Remove unused "$className" argument
This commit is contained in:
Marco Pivetta 2017-01-12 17:04:01 +01:00 committed by GitHub
commit 31c40f8342
4 changed files with 5 additions and 6 deletions

View File

@ -1,5 +1,9 @@
# Upgrade to 2.5
## Minor BC BREAK: removed $className parameter on `AbstractEntityInheritancePersister#getSelectJoinColumnSQL`
As `$className` parameter was not used in the method, it was safely removed.
## Minor BC BREAK: query cache key time is now a float
As of 2.5.5, the `QueryCacheEntry#time` property will contain a float value

View File

@ -82,12 +82,11 @@ abstract class AbstractEntityInheritancePersister extends BasicEntityPersister
/**
* @param string $tableAlias
* @param string $joinColumnName
* @param string $className
* @param string $type
*
* @return string
*/
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $className, $type)
protected function getSelectJoinColumnSQL($tableAlias, $joinColumnName, $type)
{
$columnAlias = $this->getSQLColumnAlias($joinColumnName);

View File

@ -472,7 +472,6 @@ class JoinedSubclassPersister extends AbstractEntityInheritancePersister
$columnList[] = $this->getSelectJoinColumnSQL(
$tableAlias,
$srcColumn,
$className,
PersisterHelper::getTypeOfColumn(
$mapping['sourceToTargetKeyColumns'][$srcColumn],
$targetClass,
@ -521,7 +520,6 @@ class JoinedSubclassPersister extends AbstractEntityInheritancePersister
$columnList[] = $this->getSelectJoinColumnSQL(
$tableAlias,
$srcColumn,
$className,
PersisterHelper::getTypeOfColumn(
$mapping['sourceToTargetKeyColumns'][$srcColumn],
$targetClass,

View File

@ -87,14 +87,12 @@ class SingleTablePersister extends AbstractEntityInheritancePersister
continue;
}
$className = isset($assoc['inherited']) ? $assoc['inherited'] : $this->class->name;
$targetClass = $this->em->getClassMetadata($assoc['targetEntity']);
foreach ($assoc['targetToSourceKeyColumns'] as $srcColumn) {
$columnList[] = $this->getSelectJoinColumnSQL(
$tableAlias,
$srcColumn,
$className,
PersisterHelper::getTypeOfColumn(
$assoc['sourceToTargetKeyColumns'][$srcColumn],
$targetClass,