Bug #37541

Declaration of tx_rtehtmlarea_base::drawRTE() should be compatible

Added by Thomas Deinhamer 12 months ago. Updated 11 months ago.

Status:Resolved Start date:2012-05-28
Priority:Must have Due date:
Assignee:Stanislas Rolland % Done:

100%
Category:rtehtmlarea
Target version:4.5.17
TYPO3 Version:6.0 Complexity:no-brainer
PHP Version:5.4
Votes: 0

Description

PHP Runtime Notice: Declaration of tx_rtehtmlarea_base::drawRTE() should be compatible with t3lib_rteapi::drawRTE(&$pObj, $table, $field, $row, $PA, $specConf, $thisConfig, $RTEtypeVal, $RTErelPath, $thePidValue) in /typo3/6.0.0-dev/typo3/sysext/rtehtmlarea/class.tx_rtehtmlarea_base.php line 36

Associated revisions

Revision b7ea2673
Added by Stanislas Rolland 11 months ago

[BUGFIX] Declaration of tx_rtehtmlarea_base::drawRTE() not compatible

Solution: First parameter should be defined as reference so as to be
compatible with the definition of t3lib_rteapi::drawRTE().

Change-Id: I11a24893f9fb1020fb194c7d2cff2731f7ce52f0
Releases: 4.5, 4.6, 4.7, 6.0
Resolves: #37541
Reviewed-on: http://review.typo3.org/12426
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland

Revision 14a29468
Added by Stanislas Rolland 11 months ago

[BUGFIX] Declaration of tx_rtehtmlarea_base::drawRTE() not compatible

Solution: First parameter should be defined as reference so as to be
compatible with the definition of t3lib_rteapi::drawRTE().

Change-Id: I69e7e73efdfcc259f55731accb9fd429bd9996ab
Releases: 4.5, 4.6, 4.7, 6.0
Resolves: #37541
Reviewed-on: http://review.typo3.org/12427
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland

Revision 56c98378
Added by Stanislas Rolland 11 months ago

[BUGFIX] Declaration of tx_rtehtmlarea_base::drawRTE() not compatible

Solution: First parameter should be defined as reference so as to be
compatible with the definition of t3lib_rteapi::drawRTE().

Change-Id: I16adc2a616f80e867c7986319f7ffeb69f5f160c
Releases: 4.5, 4.6, 4.7, 6.0
Resolves: #37541
Reviewed-on: http://review.typo3.org/12428
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland

Revision 885d256e
Added by Stanislas Rolland 11 months ago

[BUGFIX] Declaration of tx_rtehtmlarea_base::drawRTE() not compatible

Solution: First parameter should be defined as reference so as to be
compatible with the definition of t3lib_rteapi::drawRTE().

Change-Id: Id1617a2345a6c6f564a9f8a7d6b0ac7f0cc34be8
Releases: 4.5, 4.6, 4.7, 6.0
Resolves: #37541
Reviewed-on: http://review.typo3.org/12430
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland

Revision dc5789c9
Added by Stanislas Rolland 10 months ago

[BUGFIX] Declaration of tx_rtehtmlarea_base::drawRTE() not compatible

Solution: First parameter should be defined as reference so as to be
compatible with the definition of t3lib_rteapi::drawRTE().

Change-Id: I11a24893f9fb1020fb194c7d2cff2731f7ce52f0
Releases: 4.5, 4.6, 4.7, 6.0
Resolves: #37541
Reviewed-on: http://review.typo3.org/12426
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland

History

#1
Updated by Jigal van Hemert 12 months ago

  • Category set to rtehtmlarea
  • Status changed from New to Accepted
  • Complexity set to no-brainer

I guess that only the first parameter should be turned into a reference.

#2
Updated by Franz Koch 11 months ago

same issue in Typo3 4.7.1 and php 5.4.4. And yes, defining the first parameter as reference solves the issue

#3
Updated by Gerrit Code Review 11 months ago

  • Status changed from Accepted to Under Review

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

#4
Updated by Gerrit Code Review 11 months ago

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

#5
Updated by Stanislas Rolland 11 months ago

  • Assignee set to Stanislas Rolland
  • Target version changed from 6.0.0-alpha2 to 4.5.17

#6
Updated by Stanislas Rolland 11 months ago

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

#7
Updated by Gerrit Code Review 11 months ago

  • Status changed from Resolved to Under Review

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

#8
Updated by Gerrit Code Review 11 months ago

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

#9
Updated by Stanislas Rolland 11 months ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF