Magritte-XMLBinding causes DNU when using Seaside editor example

Hello,

Magritte-XMLBinding (loaded by MagritteAddons) appears to go bad when you use descriptions of descriptions.
Try to add a description using the MADescriptionEditor example in Seaside. I get the following:

“MessageNotUnderstoord: MXXmlNamedElementTarget>>attributeName”.

Is it just me?

Note that when we unloaded the xmlbinding (no idea what it is for) the
same operation worked.
I do not really understand why a xml binding would have such side effect.

Le 9/9/15 17:51, Skip a écrit :
> Hello,
>
> Magritte-XMLBinding (loaded by MagritteAddons) appears to go bad when you use descriptions of descriptions.
> Try to add a description using the MADescriptionEditor example in Seaside. I get the following:
>
> “MessageNotUnderstoord: MXXmlNamedElementTarget>>attributeName”.
>
> Is it just me?
>

It breaks for me because of the added magritteDescriptions to MADescription and MAElementDescription.
Both MADescription>>descriptionXmlElementName and MAElementDescription>>descriptionAttributeName.

In the first case, the accessor of the description tries to get the elementName of its xmlTarget (don’t know if that makes sense). The xmlTarget in turn says “subclassResponsibility”.

In the second case, it tries to get the attributeName of the xmlTarget. I get a DNU like I said earlier.
Again, something is wrong with “xmlTarget”. This one can be triggered using the editor.

On 09-09-15 17:51, Skip wrote:
> Hello,
>
> Magritte-XMLBinding (loaded by MagritteAddons) appears to go bad when you use descriptions of descriptions.
> Try to add a description using the MADescriptionEditor example in Seaside. I get the following:
>
> “MessageNotUnderstoord: MXXmlNamedElementTarget>>attributeName”.
>
> Is it just me?

How are you loading what combination of versions of what in what version
of Pharo?
Do the tests run green, or is there already a problem with that?
I don't know anyone using the MADescriptionEditor, does that still work
with other descriptions?

The ConfigurationOfMagritte3Addons does not refer to the newest version
of Magritte-XMLBinding. (86, from 2014-4-4) but 85, from 2012-3-29. The
latest version was created by Norbert Hartl. The part of Magritte3Addons
that we use is JSON, so our updates have been restricted to that.

Stephan

On Sep 10, 2015, at 10:53, Stephan Eggermont <[hidden email]> wrote:

> How are you loading what combination of versions of what in what version of Pharo?

I will try from a clean image, latest version of Pharo 4 (#40621). I do the following:

1) Install MagritteMagic Stable using Configuration browser (I needed #asComponent to experiment with some things).
- This will install also Seaside, Magritte, etc.
2) Seaside Control Panel, add ZnZincServerAdaptor, start the server.
3) Point browser to http://localhost:8080/magritte/editor.
4) Click “Add” button.
5) View exception.

Perhaps I should have just installed the Seaside group of Magritte3’s ConfigurationOf…

> Do the tests run green, or is there already a problem with that?

1) Open Test runner
2) Select all tests with “Magritte” in it.
3) 1922 run, 1898 passes, 0 skipped, 0 expected failures, 1 failures, 23 errors, 0 unexpected passes.
Test in failure: MXXmlPersonTargetTest>>#testContainerElementCustom
Test in error: All of MAStraitMementoTest (22 tests), and MXXmlPersonReferenceTest>>#testMultipleOption
 
> I don't know anyone using the MADescriptionEditor, does that still work with other descriptions?

I’m new so I’m currently breaking a lot. :P