nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Required Items in DCT

begood321

After selecting any item within a container how do you make all items within the same container required when user tries to save DCR.

Any help is greatly appreciated.

Thanks.

Find more posts tagged with

Comments

Adam Stoller

After selecting any item within a container how do you make all items within the same container required when user tries to save DCR.

Any help is greatly appreciated.

Thanks.

Read the FormAPI manual.
If that isn't sufficient - learn to program in Javascript or hire someone who's already proficient.

Migrateduser

After selecting any item within a container how do you make all items within the same container required when user tries to save DCR.

Any help is greatly appreciated.

Thanks.

why not just make it a replicant (and if you only need 1 just make it default=0 min=0 max=1) then set all the items inside it to required.. they will all be required if the user chooses to enter the data expanding the replicant and not be required otherwise..

begood321

Thanks jrandallnyc I'll try your suggestion.

kaalraa

After selecting any item within a container how do you make all items within the same container required when user tries to save DCR.

Any help is greatly appreciated.

Thanks.

You need to put a formAPI code on all of the items change event. Once captured, you can put all others as required.

HTH

Migrateduser

You need to put a formAPI code on all of the items change event. Once captured, you can put all others as required.

HTH

I would say that depends on what he is actually trying to do, although FormAPI will certainly work and is an acceptable approach, it may just overcomplicate a simple situation if all he needs is a container with required fields, but only when someone tries to use it..

an example could be container holding 4 fields:

- image-path
- image-alt
- image-height
- image-width

although you could certainly attached FormAPI onchange events to the image-path field to set the other 3 required.. using the OOTB container/replicant of min=0 max=1 means you have 1 line of code (well 1 attribute set in the XML of the container) rather then mulitple lines of code..

with the FormAPI approach you also would need to then check on "Edit mode" (as opposed to a new DCR), if they should also be set to required, and on clearing of the value, clearing the required flag of the other fields.. and which field becomes the "owner" of being required..

so depending on the OP's requirement, from what it sounds like, just using the replicant of 1 is probably the simpler solution..

kaalraa

Thanks for a detailed explanation and I personally agree to you. I wish the example would definitely help begood321 to consider his requirements.

begood321

Thanks Guys, I got it working 90% but I still can't get code to work with containers 2-9. It just work in original container, see code below.

Thanks for any help you can give me.

function init()
{

IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/x","onItemChange",imageSize);
IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/y","onItemChange",imageSize);
IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/radius","onItemChange",imageSize);

alert ("Hello world!");

}

function imageSize()
{

var x = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/x");
//var xval = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/x").getValue();

var y = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/y");
//var yval = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/y").getValue();

var radius = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/radius");
//var radiusval = IWDatacapture.getItem("/featuredimagegroup[1]/featuredimage[1]/radius").getValue();

if (x != "")
{
y.setRequired(true);
radius.setRequired(true);
}

else if (y != "")
{
x.setRequired(true);
radius.setRequired(true);
}

else (radius != "")
{
y.setRequired(true);
x.setRequired(true);
}
}

IWEventRegistry.addFormHandler("onFormInit", init);

Adam Stoller

Your event handler functions are implicitly passed the item for which the event is being triggered - you are currently not taking advantage of that, instead you are hard-coding an explicit path to the ...[1] item.

If you hard-code it - it isn't going to do anything except what you hard-coded it to.

Try searching the forums for code samples dealing with replicant event handler functions - you should be able to find a number of examples that show the correct way to base the design of your functions.

begood321

I even tried for loop (i=1,i<=9,i++ replacing [1] with in function imageSize() and it still didn't work.

Migrateduser

although I cant say I agree with the approach.. I dont exactly know your requirements.. but I think your giving yourself more work then needed.. your not addressing making the fields required on Edit.. and then you will need to loop thru all the fields on load.. so aside from my own thoughts on how it "should" be done...

something like this should get your example working or at least continuing along your direction.

function init()
{

 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/x","onItemChange",imageSize);
 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/y","onItemChange",imageSize);
 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/radius","onItemChange",imageSize);

}

function imageSize(item)
{

 var x = IWDatacapture.getItem(item.getName() + "/../x");
 var xval = x.getValue();

 var y = IWDatacapture.getItem(item.getName() + "/../y");
 var yval = y.getValue();

 var radius = IWDatacapture.getItem(item.getName() + "/../radius");
 var radiusval = radius.getValue();

 if (xval != "" || yval != "" || radiusval != "" )
 {
  x.setRequired(true);
  y.setRequired(true);
  radius.setRequired(true);
 } 
}

IWEventRegistry.addFormHandler("onFormInit", init);

my recomendation based on your itemHandlers would be something along the lines of:

[html]

Featured Image

X

Y

Radius

[/html]

begood321

Thanks for your reply but I need to have 9 containers that works with this code. Here is my code in the DCT file attachment.

Migrateduser

Thanks for your reply but I need to have 9 containers that works with this code. Here is my code in the DCT file attachment.

so just change all the:

[html][/html]
to:
[html][/html]

and forget about all the formapi

begood321

I cannot forget about formapi, because it should work where if a user clicks 1 of the fields centerx, centery or radius the other 2 should be required. It works in original container, but not when I add another container after loading DCT.

nipper

No, you have a container of 1-9 items. Minimum of 1, max of 9. Make everything required but let the user delete entries if they only have 4.

No formapi needed (not that this is difficult, but obviously you are not getting it)

Migrateduser

lets try another approach, and I HIGHLY encourage you to read the manuals and attend some IWOV training:

replace your code (that you put as your DCT attachment) with the following:

[html]
Featured Image Group
Group of featured images.

Featured Image

CenterX

CenterY

Radius

[/html]

nipper

lets try another approach, and I HIGHLY encourage you to read the manuals and attend some IWOV training:

replace your code (that you put as your DCT attachment) with the following:

I do not think you can have a default(0) < the min(1).

Migrateduser

I do not think you can have a default(0) < the min(1).

yeah.. duh.. edited and fixed..

begood321

Thanks Guys, but I need to implement as per requirement. I cannot make all required, this must be optional for other 2 items if 1 item in this container is selected. This should be effective for all 1-9 containers, not just in the 1st container. When I get it working I'll post the solution.

Migrateduser

Thanks Guys, but I need to implement as per requirement. I cannot make all required, this must be optional for other 2 items if 1 item in this container is selected. This should be effective for all 1-9 containers, not just in the 1st container. When I get it working I'll post the solution.

OK, then your requirements are just not clear (at least not to me).. can you post exactly what your trying to do.. I am sure some version of the formAPI code I posted can be used..

begood321

Ok jrandallnyc here is the full requirement in the attached spreadsheet. Thanks for your help.

nipper

Well you have 2 choices. The right way is FormAPI. This should not be that hard.

Your second choice is to create another container that contains X, Y and Radius. This container defaults to 0 entries and has a max of 1. If someone needs to add size, then they add to the container and fill out the required entries.

Migrateduser

Well you have 2 choices. The right way is FormAPI. This should not be that hard.

Your second choice is to create another container that contains X, Y and Radius. This container defaults to 0 entries and has a max of 1. If someone needs to add size, then they add to the container and fill out the required entries.

you really think formAPI is the "right way"??? I would have thought like this makes more sense (except for the nested containers which sometimes looks bad in the UI)..

[html]
Featured Image Group
Group of featured images.

Featured Image

Source

Title

URL

CenterX

CenterY

Radius

[/html]

nipper

That is *certainly* wrong. coords should only have a min of 0 and a max of 1, you have a max of 9.

IMHO, the parameters should be in 1 container unless you have differing counts. Here we are adding a container to get out of writing workign FormAPI. The container is not needed. Now, that being said, it is certainly an easy approach, which is why I suggested it.

Andy

Migrateduser

That is *certainly* wrong. coords should only have a min of 0 and a max of 1, you have a max of 9.

IMHO, the parameters should be in 1 container unless you have differing counts. Here we are adding a container to get out of writing workign FormAPI. The container is not needed. Now, that being said, it is certainly an easy approach, which is why I suggested it.

Andy

i was reading x/y/radius like an image map (since it said hotspots), where you would have mulitple coords per image.. but maybe that was my overreading someone elses requirements.. if it is in fact as you say.. then I agree with you.. if it's multiple "coords" per image (which now looking at those requirements seems to be).. then I stick to my replicant answer..

and unless I am missing something.. my original suggestion of the FormAPI should have been in the correct direction.. it should handle replicants just fine.. I have not tested the below.. but something like this should work.:


function init()
{

 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/x","onItemChange",imageSize);
 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/y","onItemChange",imageSize);
 IWEventRegistry.addItemHandler("/featuredimagegroup/featuredimage/radius","onItemChange",imageSize);
 if (IWDCRInfo.getDCRName() != ""){
  recheckOnEdit('/featuredimagegroup');
 }  
}

function imageSize(item)
{
 var x = IWDatacapture.getItem(item.getName() + "/../x");
 var xval = x.getValue();

 var y = IWDatacapture.getItem(item.getName() + "/../y");
 var yval = y.getValue();

 var radius = IWDatacapture.getItem(item.getName() + "/../radius");
 var radiusval = radius.getValue();

 if (xval != "" || yval != "" || radiusval != "" )
 {
  x.setRequired(true);
  y.setRequired(true);
  radius.setRequired(true);
 }  else {
  x.setRequired(false);
  y.setRequired(false);
  radius.setRequired(false);
 }
}

function recheckOnEdit(val)
{
 var item = IWDatacapture.getItem(val);
  var itemArray = item.getChildren();
  for (var imageItem=0; imageItem<itemArray.length; imageItem++)
  {
   imageSize(IWDatacapture.getItem(itemArray[imageItem].getName()+'/x'));
  }
}

IWEventRegistry.addFormHandler("onFormInit", init);