← Back to team overview

anewt-developers team mailing list archive

Re: [Branch ~sander-sinaasappel/anewt/anewt.new.cxs] Rev 1480: [form] Extra method on forms to return all controls.

 

Hey,

I think "AnewtForm::get_controls_by_name()" or something like that would be
a better self-describing API.

    — Wouter

2009-07-20 klockan 22:47 skrev noreply@xxxxxxxxxxxxx:
> ------------------------------------------------------------
> revno: 1480
> committer: Sander van Schouwenburg <sander@xxxxxxxxxxxxx>
> branch nick: anewt.new.cxs
> timestamp: Mon 2009-07-20 17:55:38 +0200
> message:
>   [form] Extra method on forms to return all controls.
>   
>   This method is AnewtForm::_controls().
>   
>   
>   (from anewt.new.svn:89)
> modified:
>   form/form.lib.php
> 
> === modified file 'form/form.lib.php'
> --- form/form.lib.php	2009-07-20 15:55:01 +0000
> +++ form/form.lib.php	2009-07-20 15:55:38 +0000
> @@ -579,6 +579,18 @@
>  	}
>  
>  	/**
> +	 * Returns all controls
> +	 *
> +	 * This method is intended for form renderer implementations only.
> +	 *
> +	 * \return
> +	 *   Array of form controls (may be empty)
> +	 */
> +	function _controls() {
> +		return $this->_controls_by_name;
> +	}
> +
> +	/**
>  	 * Check whether this form contains a file upload control (or a derived
>  	 * control). This is useful to find out which enctype the form should have.
>  	 *
> 
> 
> 
> --
> lp:~sander-sinaasappel/anewt/anewt.new.cxs
> https://code.launchpad.net/~sander-sinaasappel/anewt/anewt.new.cxs
> 
> You are subscribed to branch lp:~sander-sinaasappel/anewt/anewt.new.cxs.
> To unsubscribe from this branch go to https://code.launchpad.net/~sander-sinaasappel/anewt/anewt.new.cxs/+edit-subscription.
> 

Attachment: signature.asc
Description: Digital signature


Follow ups