Mailing-List: contact users-help@isis.apache.org; run by ezmlm
Precedence: bulk
Reply-To: users@isis.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CALJOYLHuL35XgyDDgbMNeA1V9TULM7hU4WNUBCb_WmNgiea4ow@mail.gmail.com>
References: 
 <CAG=ut6+sH_RGGFu6SSUEvKDCMiLfQYMOT8WAo94NGyMJGHqgZA@mail.gmail.com>
	<CALJOYLGjPxQ-GN48Qc6d-ZtGrMgrbNwjEvd6VUfK+NKXjd1yAw@mail.gmail.com>
	<CAG=ut6LK25W32VYor-s_zBdq9LcK+uLuv8_mFXBTv-0_1jz5HA@mail.gmail.com>
	<CAG=ut6Kt8V4UMZnXkqcPk4-0KL6JwOVQ_vY0F2-2pLAXFKyuxw@mail.gmail.com>
	<CALJOYLGOTPP_eACo7n_dAXEmyYZ1Z0rnjO5TbrZdz0MNy2a-nw@mail.gmail.com>
	<CAKxceJjE95qziX0db2fYm=Gk7wfMkQ3u-A7SznTO=OoQvoT=JQ@mail.gmail.com>
	<CALJOYLHi5iVtCherbutH8NDQhsGQr9pdZ_sRmKv6Y7MfnyyY4w@mail.gmail.com>
	<CAKxceJiRTw9-DHSexLHip1JN-H81vcruEcMkxOLf=M9=_wqKeg@mail.gmail.com>
	<CALJOYLHuL35XgyDDgbMNeA1V9TULM7hU4WNUBCb_WmNgiea4ow@mail.gmail.com>
Date: Tue, 24 Nov 2015 16:11:56 +0100
Message-ID: 
 <CAKxceJgTQPiCy1OJy35GeFuMN1Y7tnN2nN_6Wer-v_Hb2GDf7A@mail.gmail.com>
Subject: Re: collection of similar types
From: =?UTF-8?Q?Vladimir_Ni=C5=A1evi=C4=87_?= <vnisevic@gmail.com>
To: "users@isis.apache.org" <users@isis.apache.org>
Content-Type: multipart/alternative; boundary=001a113cc3663ca1d605254ac378

--001a113cc3663ca1d605254ac378
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hi Dan, yes, that's exactly the problem.

I have adapted the simple-app trying to show my problem:
https://github.com/niv0/isis

Meanwhile I understood that my integration test was wrong - I haven't used
the wrapper. Now is my problem at least consistent through WebUI and
Integration test, but I still have it.

To reproduce:

   1. Start SampleAPP Webapp
   2. Create new simple object
   3. Execute action "Plan" on simple object (will add three steps of
   specific subtype). Two steps will have a specifc properties.
   4. Exectute action "Execute" on simple object - status of first step
   change to processing, but since properties are not loaded, business logi=
c
   prevent it from doing (my problem)


Regs,Vladimir


2015-11-23 16:30 GMT+01:00 Dan Haywood <dan@haywood-associates.co.uk>:

> If I understand you correctly, sounds as if the query that you are using =
is
> only querying the supertype table, rather than doing left outer jobs for
> all subtypes.
>
> But in guessing here... perhaps you could share some code that demonstrat=
es
> the issue?
> On 23 Nov 2015 3:20 pm, "Vladimir Ni=C5=A1evi=C4=87" <vnisevic@gmail.com>=
 wrote:
>
> > Hi Dan, when I execute the action on order the steps instances are ther=
e,
> > but the values of its properites are definitely empty - I've checked it
> > thru debugging of webapp. When I open a view of single step, the values
> of
> > that step are there.
> >
> > I do not need to show additional values in a table, but have a problem
> with
> > business logic when executing an action which uses a data on single ste=
p.
> >
> > Any idea?
> > Thanks, Vladimir
> >
> >
> > 2015-11-23 14:25 GMT+01:00 Dan Haywood <dan@haywood-associates.co.uk>:
> >
> > > The objects are loaded, however the Wicket UI uses the compile time
> type
> > of
> > > the list to determine what columns to render.
> > >
> > > I could imagine us allowing some sort of hint for the framework to be
> > told
> > > to render the objects as per some other type (eg some "union interfac=
e"
> > > that combines all desired properties). But that would be a new
> feature, I
> > > think.
> > >
> > > Dan.
> > > On 23 Nov 2015 1:16 pm, "Vladimir Ni=C5=A1evi=C4=87" <vnisevic@gmail.=
com> wrote:
> > >
> > > > Hi, I have perhaps similar issue.
> > > >
> > > > I have an object (Order) which contains a list of objects can be
> > > different
> > > > type but all extend from the same superclass (Step). When I execute
> the
> > > > action on order thru wicket UI, the properties of concrete classes
> > > > StepType1 and StepType2 are not loaded. When I do the same thru
> > > > Intergration Test, the values are there?
> > > >
> > > >
> > > > *class Order{*
> > > > *.....*
> > > > *//region > steps (collection)*
> > > > *@Column(allowsNull =3D "true")*
> > > > *@Persistent(mappedBy =3D "order", defaultFetchGroup =3D "true")*
> > > > *private List<Step> steps =3D new ArrayList<>();*
> > > >
> > > > *@Collection(editing =3D Editing.DISABLED)*
> > > > *@CollectionLayout(render =3D RenderType.EAGERLY)*
> > > > *public List<Step> getSteps() {*
> > > > *    return steps;*
> > > > *}*
> > > >
> > > > *public void setSteps(final List<Step> steps) {*
> > > > *    this.steps =3D steps;*
> > > > *}*
> > > > *....*
> > > > *}*
> > > >
> > > >
> > > > *abstract class Step{*
> > > > *....*
> > > > *//region > order (property)*
> > > > *@Persistent(defaultFetchGroup =3D "true")*
> > > > *@Column(allowsNull =3D "false")*
> > > > *private Order order;*
> > > >
> > > > *@MemberOrder(name =3D "General", sequence =3D "1.1")*
> > > > *@PropertyLayout(named =3D "Order", hidden =3D Where.PARENTED_TABLE=
S)*
> > > > *public Order getOrder() {*
> > > > *    return order;*
> > > > *}*
> > > >
> > > > *public void setOrder(final Order order) {*
> > > > *    this.order =3D order;*
> > > > *}*
> > > > *.....*
> > > > *}*
> > > >
> > > >
> > > > *class StepType1 extends Step{*
> > > > *//region > ipAdresse (property)*
> > > > *private String ipAdresse;*
> > > >
> > > > *@MemberOrder(sequence =3D "1")*
> > > > *@javax.jdo.annotations.Column(allowsNull =3D "true")*
> > > > *public String getIpAdresse() {*
> > > > *    return ipAdresse;*
> > > > *}*
> > > >
> > > > *public void setIpAdresse(final String ipAdresse) {*
> > > > *    this.ipAdresse =3D ipAdresse;*
> > > > *}*
> > > > *....*
> > > > *} *
> > > >
> > > > *class StepType2 extends Step{*
> > > > *....*
> > > > *private String dslam;*
> > > >
> > > > *@MemberOrder(name =3D "Output", sequence =3D "1.0")*
> > > > *@PropertyLayout(named =3D "DSLAM")*
> > > > *@javax.jdo.annotations.Column(allowsNull =3D "true")*
> > > > *public String getDslam() {*
> > > > *    return dslam;*
> > > > *}*
> > > >
> > > > *public void setDslam(final String dslamId) {*
> > > > *    this.dslam =3D dslamId;*
> > > > *}*
> > > > *.....*
> > > > *} *
> > > >
> > > >
> > > >
> > > >
> > > > 2015-11-22 15:23 GMT+01:00 Dan Haywood <dan@haywood-associates.co.u=
k
> >:
> > > >
> > > > > On 21 November 2015 at 22:01, Stephen Cameron <
> > > > steve.cameron.62@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > Actually, it already does have the behaviour that I wanted.
> > > > > >
> > > > > >
> > > > > OK, that's good to hear.
> > > > >
> > > > >
> > > > > >
> > > > > > I'll try individual icons for the subtypes and see if it uses
> them
> > > > > instead
> > > > > > of the one for the base-type.
> > > > > >
> > > > > > It should do.  In the todoapp we use the similar cssClass() to
> > > return a
> > > > > CSS class name per instance, and then use application.css to styl=
e
> > the
> > > > > different values.  This is how individual rows within a table sho=
w
> > some
> > > > > todo items "crossed-off", ie with a line through the centre.   Th=
e
> > > icons
> > > > > also vary on a case by case basis.
> > > > >
> > > > > Cheers
> > > > > Dan
> > > > >
> > > >
> > >
> >
>

--001a113cc3663ca1d605254ac378--