Alan Pinstein schrieb:> Heh well I wouldn't know, because if you search for "this" on the > archive, it decides that "this" is a common word you don't need to > search for :( I tried!>> Hopefully they'll have a quick answer for me then.>> Alan>> On Dec 7, 2005, at 10:19 AM, David Zülke wrote:>>> I remember there was some discussion about $this->this on the >> internals list some time ago... not sure what exactly it was about >> though.>>>> - David>>>>>>>> Am 07.12.2005 um 15:41 schrieb Hans Lellelid:>>>>> Hi Alan,>>>>>> Would you be willing to write to internals PHP list with these examples>>> & see if this is something that we can rely on? If the $this->this, >>> for>>> example, is something that is not a documented language "feature" (and>>> will be taken away) then we probably don't want to rely on it, but>>> otherwise I think we should apply these fixes.>>>>>> (I can also contact them, if you prefer, but I think you have a better>>> grasp at this point of what exactly is going on here.)>>>>>> Thanks *so much* for your very thorough & dedicated attention & >>> solution>>> to this problem!>>>>>> Hans>>>>>> Alan Pinstein wrote:>>>> Hmm... well I am on 5.0.4.>>>>>>>> There are actually still leaks in some cases (like out of try/catch)>>>> but I saw the Changelog and am hoping that 5.0.5+ fixes this.>>>>>>>> But what I fixed wasn't a PHP bug; it was breaking a circular- >>>> reference>>>> deadlock that prevented refcount from ever reaching 0.>>>>>>>> Alan>>>>>>>> On Dec 7, 2005, at 2:27 AM, Sonke Ruempler wrote:>>>>>>>>> Hi,>>>>>>>>>> Alan Pinstein <mailto:apinstein​@mac.com> wrote on Wednesday, >>>>> December>>>>> 07,>>>>> 2005 2:16 AM:>>>>>>>>>>> Well, I can't believe it, but after spending all day on this,>>>>>> literally about 6 hours, I found a solution. Sample code is >>>>>> attached.>>>>>>>>>>>>>>> PHP Version?>>>>>>>>>> All versions < 5.0.5 and 4.4.0 have reference bugs.>>>>>>>>>>>>>>> --------------------​--------------------​--------------------​--------->>>>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>>>>>>> --------------------​--------------------​--------------------​--------->>>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>>> --------------------​--------------------​--------------------​--------->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>>

Heh well I wouldn't know, because if you search for "this" on the archive, it decides that "this" is a common word you don't need to search for :( I tried!

Hopefully they'll have a quick answer for me then.

Alan

On Dec 7, 2005, at 10:19 AM, David Zülke wrote:

> I remember there was some discussion about $this->this on the > internals list some time ago... not sure what exactly it was about > though.>> - David>>>> Am 07.12.2005 um 15:41 schrieb Hans Lellelid:>>> Hi Alan,>>>> Would you be willing to write to internals PHP list with these >> examples>> & see if this is something that we can rely on? If the $this- >> >this, for>> example, is something that is not a documented language >> "feature" (and>> will be taken away) then we probably don't want to rely on it, but>> otherwise I think we should apply these fixes.>>>> (I can also contact them, if you prefer, but I think you have a >> better>> grasp at this point of what exactly is going on here.)>>>> Thanks *so much* for your very thorough & dedicated attention & >> solution>> to this problem!>>>> Hans>>>> Alan Pinstein wrote:>>> Hmm... well I am on 5.0.4.>>>>>> There are actually still leaks in some cases (like out of try/catch)>>> but I saw the Changelog and am hoping that 5.0.5+ fixes this.>>>>>> But what I fixed wasn't a PHP bug; it was breaking a circular- >>> reference>>> deadlock that prevented refcount from ever reaching 0.>>>>>> Alan>>>>>> On Dec 7, 2005, at 2:27 AM, Sonke Ruempler wrote:>>>>>>> Hi,>>>>>>>> Alan Pinstein <mailto:apinstein​@mac.com> wrote on Wednesday, >>>> December>>>> 07,>>>> 2005 2:16 AM:>>>>>>>>> Well, I can't believe it, but after spending all day on this,>>>>> literally about 6 hours, I found a solution. Sample code is >>>>> attached.>>>>>>>>>>>> PHP Version?>>>>>>>> All versions < 5.0.5 and 4.4.0 have reference bugs.>>>>>>>>>>>> --------------------​--------------------​--------------------​------->>>> -->>>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>>> --------------------​--------------------​--------------------​-------->>> ->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>

I remember there was some discussion about $this->this on the internals list some time ago... not sure what exactly it was about though.

- David

Am 07.12.2005 um 15:41 schrieb Hans Lellelid:

> Hi Alan,>> Would you be willing to write to internals PHP list with these > examples> & see if this is something that we can rely on? If the $this- > >this, for> example, is something that is not a documented language "feature" (and> will be taken away) then we probably don't want to rely on it, but> otherwise I think we should apply these fixes.>> (I can also contact them, if you prefer, but I think you have a better> grasp at this point of what exactly is going on here.)>> Thanks *so much* for your very thorough & dedicated attention & > solution> to this problem!>> Hans>> Alan Pinstein wrote:>> Hmm... well I am on 5.0.4.>>>> There are actually still leaks in some cases (like out of try/catch)>> but I saw the Changelog and am hoping that 5.0.5+ fixes this.>>>> But what I fixed wasn't a PHP bug; it was breaking a circular- >> reference>> deadlock that prevented refcount from ever reaching 0.>>>> Alan>>>> On Dec 7, 2005, at 2:27 AM, Sonke Ruempler wrote:>>>>> Hi,>>>>>> Alan Pinstein <mailto:apinstein​@mac.com> wrote on Wednesday, >>> December>>> 07,>>> 2005 2:16 AM:>>>>>>> Well, I can't believe it, but after spending all day on this,>>>> literally about 6 hours, I found a solution. Sample code is >>>> attached.>>>>>>>>> PHP Version?>>>>>> All versions < 5.0.5 and 4.4.0 have reference bugs.>>>>>>>>> --------------------​--------------------​--------------------​-------->>> ->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>>

Yes, I'd be happy to. Originally I joined the php-users list to ask about the problem, but now that I've found "$this->this" I think I need to go to php-dev. I am scared to use it too! :)

Although in the meantime I'm very happy the leak is gone. I didn't like having to ini_set('memory_limit', '250M') just to load my data...

You're welcome for the help! It's nice to find a project where one can actually add something... It's not like I'm helping apache out any ;) Propel is a great project and a huge time-saver for me, so I am glad to help, although I never like spending 15 hours on one issue :(

Alan

On Dec 7, 2005, at 9:41 AM, Hans Lellelid wrote:

> Hi Alan,>> Would you be willing to write to internals PHP list with these > examples> & see if this is something that we can rely on? If the $this- > >this, for> example, is something that is not a documented language "feature" (and> will be taken away) then we probably don't want to rely on it, but> otherwise I think we should apply these fixes.>> (I can also contact them, if you prefer, but I think you have a better> grasp at this point of what exactly is going on here.)>> Thanks *so much* for your very thorough & dedicated attention & > solution> to this problem!>> Hans>> Alan Pinstein wrote:>> Hmm... well I am on 5.0.4.>>>> There are actually still leaks in some cases (like out of try/catch)>> but I saw the Changelog and am hoping that 5.0.5+ fixes this.>>>> But what I fixed wasn't a PHP bug; it was breaking a circular- >> reference>> deadlock that prevented refcount from ever reaching 0.>>>> Alan>>>> On Dec 7, 2005, at 2:27 AM, Sonke Ruempler wrote:>>>>> Hi,>>>>>> Alan Pinstein <mailto:apinstein​@mac.com> wrote on Wednesday, >>> December>>> 07,>>> 2005 2:16 AM:>>>>>>> Well, I can't believe it, but after spending all day on this,>>>> literally about 6 hours, I found a solution. Sample code is >>>> attached.>>>>>>>>> PHP Version?>>>>>> All versions < 5.0.5 and 4.4.0 have reference bugs.>>>>>>>>> --------------------​--------------------​--------------------​-------->>> ->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>

Would you be willing to write to internals PHP list with these examples& see if this is something that we can rely on? If the $this->this, forexample, is something that is not a documented language "feature" (andwill be taken away) then we probably don't want to rely on it, butotherwise I think we should apply these fixes.

(I can also contact them, if you prefer, but I think you have a bettergrasp at this point of what exactly is going on here.)

So when memory-managing objects with circular links, the normal way to do this is to have parents "retain" (keep ref-counted links) to their kids, and have the kids have "weak references" (non-ref- counted) links to their parents. This way, when the parent is no longer used, it will automatically 0-out the ref counts to all children it links too and things GC correctly.

Now, how to do this in PHP?

Well, it seems that if you create a reference to another object, it's refCount isn't bumped... see the attached weakRefTest.php for an example.

A circular reference is being created... since PHP doesn't have weak references I am not sure how to fix this... off the top of my head I don't see how this circular reference can be eliminated without losing functionality.

There seems to be a lot of talk on php lists about the circular reference problem. The php-dev guys seem to think it's not a bug, which I suppose "technically" it isn't since the memory is freed, but it is then certainly a language deficit for doing big OO projects.

Heh man we keep swapping our logic. Yes, what you said is what I meant. I need more sleep.

Alan

On Dec 6, 2005, at 11:34 AM, Ron Rademaker wrote:

> Well, if noxsd='true', the schema shouldn't be used (so there's no > need> for the schema to support it). If noxsd is anything but true, > you're not> using it right and the schema should complain.>> Ron>> On Tue, 2005-12-06 at 11:31 -0500, Alan Pinstein wrote:>> heh... I was wondering.>>>> OK, so if you don't want to validate, put in {noxsd="true"}, but>> never use {noxsd="false"}... makes sense, since this is really only>> for debugging.>>>> It was still funny :)>>>> Alan>>>> On Dec 6, 2005, at 11:29 AM, Ron Rademaker wrote:>>>>> (Except for the first one of course, noxsd should be in the xsd).>>> Should>>> be (Except for the first one of course, noxsd should not be in the>>> xsd).>>>>>> On Tue, 2005-12-06 at 17:21 +0100, Ron Rademaker wrote:>>>> I've just commited a new XSD that should take care of all your>>>> errors.>>>>>>>> (Except for the first one of course, noxsd should be in the xsd).>>>>>>>> On Tue, 2005-12-06 at 10:52 -0500, Alan Pinstein wrote:>>>>>> Maybe you could report on some of the errors you were getting?>>>>>> There>>>>>> are still some problems with the XSD complaining about valid>>>>>> schemas. I>>>>>> need to add a fix for <id-method-parameter/> to accept @name>>>>>> attrib, I know.>>>>>>>>>> Of course!>>>>>>>>>> Alan>>>>>>>>>> PS - I thoroughly enjoyed the irony of the first warning :)>>>>>>>>>> ====================​====================​=============>>>>>>>>>> Warning: Element 'database': The attribute 'noxsd' is not allowed.>>>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line>>>>> 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' >>>>> is not>>>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>>>> classes/>>>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>>>>>> Warning: Element 'id-method-parameter': The attribute >>>>> {'translate'}>>>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>>>> generator/classes/pr​opel/engine/database​/transform/>>>>> XmlToAppData.php>>>>> on line 118>>>>>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>>>> parameter']: Missing child element(s). Expected is ( rule ). in />>>>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/>>>>> engine/>>>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>>>>>>>> --------------------​--------------------​--------------------​------>>>>> -->>>>> ->>>>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>>>>>>> --------------------​--------------------​--------------------​------->>>> -->>>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>>>>>> --------------------​--------------------​--------------------​-------->>> ->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>

OK, so if you don't want to validate, put in {noxsd="true"}, but never use {noxsd="false"}... makes sense, since this is really only for debugging.

It was still funny :)

Alan

On Dec 6, 2005, at 11:29 AM, Ron Rademaker wrote:

> (Except for the first one of course, noxsd should be in the xsd). > Should> be (Except for the first one of course, noxsd should not be in the > xsd).>> On Tue, 2005-12-06 at 17:21 +0100, Ron Rademaker wrote:>> I've just commited a new XSD that should take care of all your >> errors.>>>> (Except for the first one of course, noxsd should be in the xsd).>>>> On Tue, 2005-12-06 at 10:52 -0500, Alan Pinstein wrote:>>>> Maybe you could report on some of the errors you were getting? >>>> There>>>> are still some problems with the XSD complaining about valid>>>> schemas. I>>>> need to add a fix for <id-method-parameter/> to accept @name>>>> attrib, I know.>>>>>> Of course!>>>>>> Alan>>>>>> PS - I thoroughly enjoyed the irony of the first warning :)>>>>>> ====================​====================​=============>>>>>> Warning: Element 'database': The attribute 'noxsd' is not allowed.>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>> Warning: Element 'column': The attribute 'require' is not allowed.>>> in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/>>> engine/database/tran​sform/XmlToAppData.p​hp on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'default' is>>> not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/>>> classes/propel/engin​e/database/transform​/XmlToAppData.php on line >>> 118>>>>>> Warning: Element 'id-method-parameter': The attribute 'value' is not>>> allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/>>> propel/engine/databa​se/transform/XmlToAp​pData.php on line 118>>>>>> Warning: Element 'id-method-parameter': The attribute {'translate'}>>> is required but missing. in /Users/Shared/Develo​pment/src/propel/>>> generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.php>>> on line 118>>>>>> Warning: Element 'id-method-parameter' [complex type 'id-method->>> parameter']: Missing child element(s). Expected is ( rule ). in />>> Users/Shared/Develop​ment/src/propel/gene​rator/classes/propel​/engine/>>> database/transform/X​mlToAppData.php on line 118>>>>>>>>> --------------------​--------------------​--------------------​-------->>> ->>> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>>>>>> --------------------​--------------------​--------------------​--------->> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>> For additional commands, e-mail: dev-help at propel dot tigris dot org>>>>> --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>

> Maybe you could report on some of the errors you were getting? There> are still some problems with the XSD complaining about valid > schemas. I> need to add a fix for <id-method-parameter/> to accept @name > attrib, I know.

Of course!

Alan

PS - I thoroughly enjoyed the irony of the first warning :)

====================​====================​=============

Warning: Element 'database': The attribute 'noxsd' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/engine/database/tran​sform/XmlToAppData.p​hp on line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Warning: Element 'column': The attribute 'require' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/engine/database/tran​sform/XmlToAppData.p​hp on line 118

Warning: Element 'column': The attribute 'require' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/engine/database/tran​sform/XmlToAppData.p​hp on line 118

Warning: Element 'column': The attribute 'require' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/engine/database/tran​sform/XmlToAppData.p​hp on line 118

Warning: Element 'column': The attribute 'require' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/prope​l/engine/database/tran​sform/XmlToAppData.p​hp on line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Warning: Element 'id-method-parameter': The attribute 'default' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engin​e/database/transform​/XmlToAppData.php on line 118

Warning: Element 'id-method-parameter': The attribute 'value' is not allowed. in /Users/Shared/Develo​pment/src/propel/gen​erator/classes/propel/engine/databa​se/transform/XmlToAp​pData.php on line 118

Warning: Element 'id-method-parameter': The attribute {'translate'} is required but missing. in /Users/Shared/Develo​pment/src/propel/generator/classes/pr​opel/engine/database​/transform/XmlToAppD​ata.phpon line 118

Alan Pinstein wrote:> Just a few quick notes from me as I am testing the SVN / CVS versions > of everything.> > 1. I had all kinds of schema validation problems. Adding:> > noxsd="true"> > to the <database> tag successfully skips schema validation.

Maybe you could report on some of the errors you were getting? Thereare still some problems with the XSD complaining about valid schemas. Ineed to add a fix for <id-method-parameter/> to accept @name attrib, I know.

> 2. I am still seeing memory leaks [ had to do with refcount deadlocks > with collections IIRC ] in my sync script, although definitely the > leaks are much smaller. I will have to do further testing to see if the> leaks are mine or in Propel.> > Test mem usage with propel 1.1 (10 iterations):> >> Importing: ./savedData.txt>> 953976 (before propel is used)>> 4832768>> 4944008>> 5141904>> 5351880>> 5543848>> 5846512>> 6051464>> 6305152>> 6434792> > > ~200k per iteration leak> > and with SVN:> >> Importing: ./savedData.txt>> 958504 (before propel is used)>> 4785104>> 4804072>> 4839160>> 4876072>> 4910008>> 4959096>> 4995296>> 5039896>> 5062424> > > ~34k per iteration leak> > Definitely much better! But man that's a lotta leaking. I suppose this> could get down into creole as well. It will take some digging to > isolate this. Of course, one of the nice things about PHP is that > minimal memory leaking isn't that dangerous in a web script, but it's > still a PITA in cli scripts!> > Question: will a creole release coincide with the propel 1.2? Will it > be what's in CVS presently? Just want to make sure I'm integration > testing the right stuff.>

Just a few quick notes from me as I am testing the SVN / CVS versions of everything.

1. I had all kinds of schema validation problems. Adding:

noxsd="true"

to the <database> tag successfully skips schema validation.

2. I am still seeing memory leaks [ had to do with refcount deadlocks with collections IIRC ] in my sync script, although definitely the leaks are much smaller. I will have to do further testing to see if the leaks are mine or in Propel.

Definitely much better! But man that's a lotta leaking. I suppose this could get down into creole as well. It will take some digging to isolate this. Of course, one of the nice things about PHP is that minimal memory leaking isn't that dangerous in a web script, but it's still a PITA in cli scripts!

Question: will a creole release coincide with the propel 1.2? Will it be what's in CVS presently? Just want to make sure I'm integration testing the right stuff.

I thought in the end it seemed like the return values were "ok" -- or at least there didn't seem to be a better alternative. Or maybe I'm misremembering; I think we decided that array of ValidationFailed objects on failure was good, right? -- or do we want it to return false and then have a separate method to get the ValidationFailed objects (or maybe the object's validate() method could return false and then there'd be a getValidationFailures() method or something).

& yes -- I think we should fix the non-required empty values issue. i forgot about that. Ok -- I'll try to spend some time on that this week, but feel free to tackle any of that too. I won't have time until Saturday at the earliest.

Thanks!Hans

David Zülke wrote:> I believe we agreed upon changing doValidate() so it doesn't bail out on> non-required empty values, and then there was something about more useful> return values... IIRC. Got some time this week to work on Propel if needed,> so I'm "awaiting orders" ;)> > - David> > > >>-----Original Message----->>From: Hans Lellelid [mailto:hans at velum dot net]>>Sent: Tuesday, November 22, 2005 3:51 AM>>To: dev at propel dot tigris dot org>>Subject: [propel-dev] propel 1.2>>>>Hi David et al,>>>>Was there a feeling of consensus in the end about any changes that need>>to be made to the validator framework before 1.2? I'm not sure that>>realistically I'm going to be able to spend much time on Propel in the>>next couple weeks, so I'm eager to release a release candidate for>>public consumption. It's overdue. I don't want to rush something if>>there are some changes that need to be made, though.>>>>Thanks->>Hans>>>>------------​--------------------​--------------------​----------------->>To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org>>For additional commands, e-mail: dev-help at propel dot tigris dot org>>> > > > > > --------------------​--------------------​--------------------​---------> To unsubscribe, e-mail: dev-unsubscribe@prop​el.tigris.org> For additional commands, e-mail: dev-help at propel dot tigris dot org>