Answered

Patch field only if checkbox ticked - need to press button twice for itto work, weird! please help!

(0) Share

Report

Posted on by Laura May

I am trying to create a patch that looks at the fields in a form and only executes the patch if an associated checkbox is ticked. I have tried a few ways of ordering the commands, this is as close as I seem to be able to get! This logic works if I press the button twice. First press saves it to the Clausecollection, second press patches it to clause table.

Would you show me how it needs to be changed to make everything happen in one press please?

If(

CheckboxClauseName.Value = true,

UpdateIf(ClauseCollection,true, {clauseName: TextClauseName.Text}),

Patch(clauses,ClauseCollection));

If(

CheckboxClauseBusinessDescription.Value = true,

UpdateIf(ClauseCollection,true,{clauseBusinessDescription: TextClauseBusinessDescription.Text}),

Patch(clauses,ClauseCollection));

If(

CheckboxClausePartyType.Value = true,

UpdateIf(ClauseCollection,true,{clausePartyType: ChoiceClausePartyType.Selected.Value}),

Patch(clauses,ClauseCollection));

If(

CheckboxClause_theme.Value = true,

UpdateIf(ClauseCollection,true,{clause_theme: LookupClause_Theme.Selected}),

Patch(clauses,ClauseCollection));

Categories:

Building Power Apps

I have the same question (0)

All responses (5)

Answers (1)

Sort by

eka24 20,925 on at

Like
a
(0)

Report
Copy link

Link copied!

What is not clear to me is the true inside this:

UpdateIf(ClauseCollection,true, {clauseName: TextClauseName.Text})

Do you intend checking any column?

If you can give more explanation

------------

If you like this post, give a Thumbs up. Where it solved your request, Mark it as a Solution to enable other users find it.

------------

If you like this post, give a Thumbs up. Where it solved your request, Mark it as a Solution to enable other users find it.

Was this reply helpful? Yes No
Verified answer

v-xiaochen-msft Microsoft Employee on at

Like
a
(0)

Report
Copy link

Link copied!

Hi @LauraMay ,

You should use ";" instead of "," .

You could try this formula:
If(
CheckboxClauseName.Value = true,
UpdateIf(ClauseCollection,true, {clauseName: TextClauseName.Text});
Patch(clauses,ClauseCollection));

If(
CheckboxClauseBusinessDescription.Value = true,
UpdateIf(ClauseCollection,true,{clauseBusinessDescription: TextClauseBusinessDescription.Text});
Patch(clauses,ClauseCollection));

If(
CheckboxClausePartyType.Value = true,
UpdateIf(ClauseCollection,true,{clausePartyType: ChoiceClausePartyType.Selected.Value});
Patch(clauses,ClauseCollection));

If(
CheckboxClause_theme.Value = true,
UpdateIf(ClauseCollection,true,{clause_theme: LookupClause_Theme.Selected});
Patch(clauses,ClauseCollection));

Best Regards,
Wearsky
If my post helps, then please consider Accept it as the solution to help others. Thanks.

Was this reply helpful? Yes No
Laura May 67 on at

Like
a
(0)

Report
Copy link

Link copied!

Hi all,
I got this to work using the following, basically doing all the UpdateIf first, then pushing the Patch to the end. The first section creates new records based on the ClauseCollection. The second section edits existing records based on whats in the ClauseCollection

If(GroupClauseCollection = true && CopyCLRecord = true, //Copy Multiple Records
If(CheckboxClauseName.Value = true, UpdateIf(ClauseCollection, true, {clauseName: TextClauseName.Text}));
If(CheckboxClauseBusinessDescription.Value = true, UpdateIf(ClauseCollection, true, {clauseBusinessDescription: TextClauseBusinessDescription.Text}));
If(CheckboxClausePartyType.Value = true, UpdateIf( ClauseCollection, true, {clausePartyType: ChoiceClausePartyType.Selected.Value}));
If(CheckboxClause_theme.Value = true, UpdateIf(ClauseCollection, true, {clause_theme: LookupClause_Theme.Selected}));

ClearCollect(CopyMultipleClausesCollection, DropColumns(ClauseCollection, "crb8f_clauseuid", "crb8f_clauseid", "ClauseUID" ));
Patch(clauses,CopyMultipleClausesCollection));

If(GroupClauseCollection = true && EditCLRecord = true, //Edit Multiple Records
If(CheckboxClauseName.Value = true,UpdateIf(ClauseCollection, true, {clauseName: TextClauseName.Text}));
If(CheckboxClauseBusinessDescription.Value = true,UpdateIf(ClauseCollection, true, {clauseBusinessDescription: TextClauseBusinessDescription.Text}));
If(CheckboxClausePartyType.Value = true, UpdateIf(ClauseCollection, true, {clausePartyType: ChoiceClausePartyType.Selected.Value}));
If(CheckboxClause_theme.Value = true, UpdateIf(ClauseCollection, true, {clause_theme: LookupClause_Theme.Selected}));
Patch(clauses, ClauseCollection));

Was this reply helpful? Yes No
Laura May 67 on at

Like
a
(0)

Report
Copy link

Link copied!

Hi @eka24 The presence of true inside this:
UpdateIf(ClauseCollection,true, {clauseName: TextClauseName.Text})

In all honesty, I found the syntax on a forum, I wasnt sure of the purpose of true either but it seems to work.

Was this reply helpful? Yes No
Laura May 67 on at

Like
a
(0)

Report
Copy link

Link copied!

I am now trying to create the equivalent functionality in an App embedded in PowerBi

The faster UpdateIf and Patch approach does not work in an App embedded in PowerBI so I have reverted back to ForAll and Patch approach. The below works to Edit existing records

ForAll(
Collection,
Patch(
DataSource,
LookUp(
DataSource,
crb8f_datasourceid = Collection[@crb8f_collectionid]),
{
DataSourceField1: DropDownLookUpofDataSourceField1.Selected,
DataSourceField2: DropDownChoiceofDataSourceField2.Selected.Value,
etc etc etc,
}
)
)

However when it comes to copying existing records, the equivalent Patch does not work, I think because of the difference between the patch expecting a Record and I am giving it a Table from the dropdown (something like that, I may have it the wrong way round!)

ForAll(
Collection,
Patch(
DataSource,
Defaults(DataSource),
{
DataSourceField1: DropDownLookUpofDataSourceField1.Selected,
DataSourceField2: DropDownChoiceofDataSourceField2.Selected.Value,
etc etc etc,
}
)
)
);

I got round this by trying.......

Patch(collection,
Defaults(sourcetable),
{ SourceField: LookUp(sourcetable, sourcefield in [@Collection].crb8f_CollectionField) }));

If the collection contains 4 records, all with different field content, the Patch will create 4 records, however all 4 records are crreated exactly the same, but the fields are populated based on the first record Power Apps finds in the Collection table (I guess because I am using the Lookup function) This is the best way I can explain, it.

Collection Records
Record 1: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: Field updated by user in Power Apps
Record 2: FieldA: 2a FieldB: 2b FieldC: 2c FieldD: Field updated by user in Power Apps
Record 3: FieldA: 3a FieldB: 3b FieldC: 3c FieldD: Field updated by user in Power Apps
Record 4: FieldA: 4a FieldB: 4b FieldC: 4c FieldD: Field updated by user in Power Apps

New Records Required
Record 5: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: Field updated by user in Power Apps
Record 6: FieldA: 2a FieldB: 2b FieldC: 2c FieldD: Field updated by user in Power Apps
Record 7: FieldA: 3a FieldB: 3b FieldC: 3c FieldD: Field updated by user in Power Apps
Record 8: FieldA: 4a FieldB: 4b FieldC: 4c FieldD: Field updated by user in Power Apps

New Records Created
Record 5: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: User defined field
Record 6: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: User defined field
Record 7: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: User defined field
Record 8: FieldA: 1a FieldB: 1b FieldC: 1c FieldD: User defined field

Do I need to change when I trigger the ForAll loop or something?

Was this reply helpful? Yes No