Task #34629

Clean up phpDoc comments in class template

Added by Sebastian Michaelsen over 9 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Code Cleanup
Target version:
-
Start date:
2012-03-07
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.7
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

In analogy to #28087 I want to clean up the template class (typo3/template.php) this time. Clean phpDoc comments are important to make the API understandable to Users and IDEs.

So I made a patch doing mostly:

  • Provide correct parameter names in phpDoc
  • Provide correct data types in phpDoc
  • CGLify phpDoc (remove tabs)
  • CGLify method signatures (the first line of the method)
  • Corrected a few minor very obvious flaws ("$size = $size;" was my favorite)

Note on green bar feeling: After the patch the class is not green yet, but it has improved a lot. I'd like to tackle a few more flaws when this obvious changes are through.


Related issues

Related to TYPO3 Core - Bug #34882: Clean up phpDoc comments in t3lib_TStemplateClosed2012-03-15

Actions
Related to TYPO3 Core - Task #28087: Clean up phpDoc comments in t3lib_divClosed2011-07-11

Actions
#1

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

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

#2

Updated by Gerrit Code Review over 9 years ago

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

#3

Updated by Gerrit Code Review over 9 years ago

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

#4

Updated by Gerrit Code Review about 9 years ago

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

#5

Updated by Sebastian Michaelsen about 9 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF