Google Mail Calendar Documents Reader Web more »
Recently Visited Groups | Help | Sign in
Google Groups Home
#1652: New constant 'Doctrine::VALIDATE_USER'
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  6 messages - Collapse all  -  Translate all to Translated (View all originals)
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
 
From:
To:
Cc:
Follow-up To:
Add Cc | Add Follow-up to | Edit Subject
Subject:
Validation:
For verification purposes please type the characters you see in the picture below or the numbers you hear by clicking the accessibility icon. Listen and type the numbers that you hear
 
Doctrine  
View profile   Translate to Translated (View Original)
 More options 7 Nov 2008, 10:45
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Fri, 07 Nov 2008 10:45:08 -0000
Local: Fri 7 Nov 2008 10:45
Subject: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
-----------------------------------+--------------------------------------- -
 Reporter:  floriank               |       Owner:  jwage  
     Type:  enhancement            |      Status:  new    
 Priority:  minor                  |   Milestone:  1.1.0  
Component:  Attributes             |     Version:  1.1-DEV
 Keywords:  validation             |    Has_test:  1      
 Mystatus:  Pending Core Response  |   Has_patch:  1      
-----------------------------------+--------------------------------------- -
 I'm missing a constant to control validation process in Doctrine to only
 check self-written Record->validate() methods without validating datatypes
 / lengths / constraints (see testcase).

 A possible solution would be to add a constant Doctrine::VALIDATE_USER
 (see patch). This patch breaks none of the testcases in Doctrine 1.1's
 testsuite.

--
Ticket URL: <http://trac.doctrine-project.org/ticket/1652>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Doctrine  
View profile   Translate to Translated (View Original)
 More options 7 Nov 2008, 21:38
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Fri, 07 Nov 2008 21:38:32 -0000
Local: Fri 7 Nov 2008 21:38
Subject: Re: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
--------------------------+------------------------------------------------ -
  Reporter:  floriank     |       Owner:  jwage                
      Type:  enhancement  |      Status:  new                  
  Priority:  minor        |   Milestone:  1.1.0                
 Component:  Attributes   |     Version:  1.1-DEV              
Resolution:               |    Keywords:  validation          
  Has_test:  1            |    Mystatus:  Pending Core Response
 Has_patch:  1            |  
--------------------------+------------------------------------------------ -
Comment (by romanb):

 You can already disable length/type validations. I dont remember the
 constant names right now. SOmething like
 ATTR_VALIDATE_LENGTH/ATTR_VALIDATE_TYPE. Is that not sufficient?

--
Ticket URL: <http://trac.doctrine-project.org/ticket/1652#comment:1>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Doctrine  
View profile   Translate to Translated (View Original)
 More options 7 Nov 2008, 22:06
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Fri, 07 Nov 2008 22:06:20 -0000
Local: Fri 7 Nov 2008 22:06
Subject: Re: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
--------------------------+------------------------------------------------ -
  Reporter:  floriank     |       Owner:  jwage                
      Type:  enhancement  |      Status:  new                  
  Priority:  minor        |   Milestone:  1.1.0                
 Component:  Attributes   |     Version:  1.1-DEV              
Resolution:               |    Keywords:  validation          
  Has_test:  1            |    Mystatus:  Pending Core Response
 Has_patch:  1            |  
--------------------------+------------------------------------------------ -
Comment (by floriank):

 Replying to [comment:1 romanb]:
 > I dont remember the constant names right now.
 It is
 {{{
 Doctrine_Manager::getInstance()->setAttribute(Doctrine::ATTR_VALIDATE,
 Doctrine::VALIDATE_ALL &
 ~Doctrine::VALIDATE_LENGTHS &
 ~Doctrine::VALIDATE_CONSTRAINTS &
 ~Doctrine::VALIDATE_TYPES);

 }}}

 I want to disable all validation except the validate()-method of the
 Records, which is not possible because the code above is the same as

 {{{
 Doctrine_Manager::getInstance()->setAttribute(Doctrine::ATTR_VALIDATE,
 Doctrine::VALIDATE_NONE);
 }}}

 what results in a static return of true in Record->isValid().

--
Ticket URL: <http://trac.doctrine-project.org/ticket/1652#comment:2>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Doctrine  
View profile   Translate to Translated (View Original)
 More options 1 Dec 2008, 19:37
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Mon, 01 Dec 2008 19:37:21 -0000
Local: Mon 1 Dec 2008 19:37
Subject: Re: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
-----------------------------------+--------------------------------------- -
 Reporter:  floriank               |       Owner:  jwage  
     Type:  enhancement            |      Status:  new    
 Priority:  minor                  |   Milestone:  1.1.0  
Component:  Attributes             |     Version:  1.1-DEV
 Keywords:  validation             |    Has_test:  1      
 Mystatus:  Pending Core Response  |   Has_patch:  1      
-----------------------------------+--------------------------------------- -

Comment(by jwage):

 I don't understand your patch. It adds the constant but does not use it
 anywhere?

--
Ticket URL: <http://trac.doctrine-project.org/ticket/1652#comment:3>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Doctrine  
View profile   Translate to Translated (View Original)
 More options 1 Dec 2008, 19:38
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Mon, 01 Dec 2008 19:38:13 -0000
Local: Mon 1 Dec 2008 19:38
Subject: Re: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
-----------------------------------+--------------------------------------- -
 Reporter:  floriank               |       Owner:  jwage  
     Type:  enhancement            |      Status:  new    
 Priority:  minor                  |   Milestone:  1.2.0  
Component:  Attributes             |     Version:  1.1-DEV
 Keywords:  validation             |    Has_test:  1      
 Mystatus:  Pending Core Response  |   Has_patch:  1      
-----------------------------------+--------------------------------------- -
Changes (by jwage):

  * milestone:  1.1.0 => 1.2.0

--
Ticket URL: <http://trac.doctrine-project.org/ticket/1652#comment:4>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Doctrine  
View profile   Translate to Translated (View Original)
 More options 2 Dec 2008, 07:35
From: "Doctrine" <no-re...@phpdoctrine.org>
Date: Tue, 02 Dec 2008 07:35:23 -0000
Local: Tues 2 Dec 2008 07:35
Subject: Re: [Doctrine] #1652: New constant 'Doctrine::VALIDATE_USER'
#1652: New constant 'Doctrine::VALIDATE_USER'
-----------------------------------+--------------------------------------- -
 Reporter:  floriank               |       Owner:  jwage  
     Type:  enhancement            |      Status:  new    
 Priority:  minor                  |   Milestone:  1.2.0  
Component:  Attributes             |     Version:  1.1-DEV
 Keywords:  validation             |    Has_test:  1      
 Mystatus:  Pending Core Response  |   Has_patch:  1      
-----------------------------------+--------------------------------------- -

Comment(by floriank):

 Replying to [comment:3 jwage]:
 > I don't understand your patch. It adds the constant but does not use it
 anywhere?
 Take a look at my testcase: It's used in there ([http://trac.doctrine-
 project.org/attachment/ticket/1652/1652TestCase.php#L53]). The constant is
 not used anywhere else in Doctrine since it is an ''additional'' option
 for validation to only run user-defined validate()-methods without beeing
 forced to use type/length/constraint-validations.

--
Ticket URL: <http://trac.phpdoctrine.org/ticket/1652#comment:5>
Doctrine <http://www.phpdoctrine.org>
PHP Doctrine Object Relational Mapper


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2009 Google