Project

General

Profile

Actions

Bug #26660

closed

imagecopyresized has invalid function parameter

Added by Benjamin Schulte almost 13 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2011-05-06
Due date:
% Done:

100%

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

Description

The class t3lib_stdgraphic provides the method imagecopyresized with a parameter list which defines the parameters $w and $h twice.

Due to this, it is impossible to create a mock of the stdGraphic class using PHPUnit.

In any way, I don't know how the method should work correctly.

function imagecopyresized(&$im, $cpImg, $Xstart, $Ystart, $cpImgCutX, $cpImgCutY, $w, $h, $w, $h)

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #36947: GIFBUILDER render twiceClosedTYPO3 Release Team2012-05-08

Actions
Related to TYPO3 Core - Bug #39563: Regression in t3lib_iconWorks::imagecopyresizedClosed2012-08-06

Actions
Actions #1

Updated by Mr. Hudson almost 13 years ago

Patch set 1 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #2

Updated by Stefan Neufeind almost 13 years ago

Quickly hacked together script to check for duplicate params. Detected two occurences of this here and didn't find something else on Core, extbase, fluid or FLOW3.

  1. search, leave out some comment-lines
    $ grep "function " -R * | grep -v \* >found

find-script:
$fp = fopen("found", "r");
while($s = fgets($fp)) {
$params = $s;
$pos = strpos($params, '(');
if($pos <= 0) continue;
$params = substr($params, $pos+1);

$pos = strpos($params, ')');
if($pos <= 0) continue;
$params = substr($params, 0, $pos);
$aParams = explode(',', $params);
$aParams = array_map('trim', $aParams);
if(count($aParams) != count(array_unique($aParams))) {
var_dump($s);
}
}
Actions #3

Updated by Mr. Hudson over 12 years ago

Patch set 2 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #4

Updated by Thorsten Kahler over 12 years ago

  • Status changed from New to Needs Feedback
  • Assignee set to Benjamin Schulte

Does this grievance really produce any errors? From what I read from PHPDoc this is just a wrapper method for internal use and the duplicated variable names always should be assigned the same values.

Actions #5

Updated by Mr. Jenkins over 12 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 3 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #6

Updated by Mr. Jenkins over 12 years ago

Patch set 1 of change I00166cb30b278e1124675111d2cc3174811e4151 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #7

Updated by Mr. Jenkins over 12 years ago

Patch set 1 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/6831

Actions #8

Updated by Mr. Jenkins over 12 years ago

Patch set 1 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/6834

Actions #9

Updated by Mr. Jenkins over 12 years ago

Patch set 4 of change Id43229caa3694e8859f9912946c963f907f92951 has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #10

Updated by Gerrit Code Review over 12 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #11

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch TYPO3_4-4 has been pushed to the review server.
It is available at http://review.typo3.org/6834

Actions #12

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/6831

Actions #13

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #14

Updated by Gerrit Code Review over 12 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #15

Updated by Gerrit Code Review over 12 years ago

Patch set 3 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #16

Updated by Gerrit Code Review over 12 years ago

Patch set 3 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/6831

Actions #17

Updated by Gerrit Code Review over 12 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/2564

Actions #18

Updated by Gerrit Code Review about 12 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/9608

Actions #19

Updated by Stefan Neufeind about 12 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #20

Updated by Gerrit Code Review about 12 years ago

  • Status changed from Resolved to Under Review

Patch set 4 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #21

Updated by Gerrit Code Review about 12 years ago

Patch set 5 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #22

Updated by Gerrit Code Review over 11 years ago

Patch set 6 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/6829

Actions #23

Updated by Gerrit Code Review over 11 years ago

Patch set 4 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/6831

Actions #24

Updated by Stefan Neufeind over 11 years ago

  • Status changed from Under Review to Resolved
Actions #25

Updated by Gerrit Code Review over 11 years ago

  • Status changed from Resolved to Under Review

Patch set 5 for branch TYPO3_4-5 has been pushed to the review server.
It is available at http://review.typo3.org/6831

Actions #26

Updated by Stefan Neufeind over 11 years ago

  • Status changed from Under Review to Resolved
Actions #27

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF