Add class type hinting to all our OO code, and array type hinting where valuable [#560274]

Type hinting will make clear to the programmer how to use the methods and will make aberrant uses caught more quickly before they lead to other sneaky errors.

http://us3.php.net/manual/en/language.oop5.typehinting.php

Comment	File	Size	Author
#1	type_hinting.patch	21.94 KB	falcon
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

falcon CreditAttribution: falcon commented 28 August 2009 at 08:10

File	Size
type_hinting.patch	21.94 KB

The php type hinting isn't very useful to us IMO. Hopefully php will support type hinting for primitive types in the future. Attached is a patch showing how type hinting has been added to the question-types.

Comment #2

falcon CreditAttribution: falcon commented 28 August 2009 at 08:27

Status:

Active

» Fixed

Comment #3

turadg CreditAttribution: turadg commented 31 August 2009 at 16:41

Status:

Fixed

» Needs work

I'm reopening this since there are a lot more signatures that can use it.

Unfortunately PHP probably won't ever have type hinting for scalars. We could start using asserts though. They don't cost performance in production (assuming assertion checking is turned off) and they act as type documentation.

Comment #4

falcon CreditAttribution: falcon commented 6 November 2009 at 14:26

Version:	6.x-4.x-dev	» 6.x-5.x-dev
Status:	Needs work	» Postponed

Comment #5

djdevin

Philadelphia

CreditAttribution: djdevin commented 15 July 2016 at 20:57

Component:	Code	» Code - Quiz core
Issue summary:	View changes
Status:	Postponed	» Closed (outdated)

This issue is being closed because it was filed against a version that is no longer supported. If the issue still persists in the latest version of Quiz, please open a new issue.