Added test for LENGTH operand

[wjzijderveld]

In the test I compare a length literally (=121) and with greater and smaller than.

Is there a PHPCR namespace URI we can use? I now just made something up and most likely wrong :)

[dbu]

what do you create this namespace for? i don't see it used anywhere.

indeed this will cause major pain with jackrabbit, as namespaces are global and we use the phpcr namespace already.

[wjzijderveld]

Added it because it caused an error about a invalid XML while I was testing this. But figured that the namespace isn't needed in the general JCR XML. Only in the XML used by jackalope doctrine dbal.

Removed it.

[dbu]

great. ping when you did that.

[dbu]

@wjzijderveld we are trying to release jackalope 1.1 - do you have time to finalize this?

[wjzijderveld]

I probably can make time for it tomorrow evening.

On Tue, Jan 7, 2014 at 9:13 PM, David Buchmann notifications@github.com
wrote:

@wjzijderveld we are trying to release jackalope 1.1 - do you have time to finalize this?

Reply to this email directly or view it on GitHub:
#123 (comment)

[wjzijderveld]

@dbu I updated the tests.
Maybe the empty-property tests are a bit too much now, Can update it to just test if a = 0 comparison works? As the > is already tested with the string test.

[dbu]

thanks @wjzijderveld i think the < 1 is fine too, does not hurt to have a few edge cases in the tests.

[dbu]

this does not work in jackalope-jackrabbit, see #127

@wjzijderveld if you could have a look, that would be great.