cancel
Showing results for 
Search instead for 
Did you mean: 
Reply
JB0007
Helper V
Helper V

Multiple Combination of If Statements

Hi Power Apps community, 

 

I am upgrading my app for more efficiency.  The problem is that there are employees that cannot go on vacation the same date. If an employee request a vacation date that has already been chosen by a specific employee, it needs to notify the employee. The rule is not applicable for ALL employees.  I have insert my code in the OnSelect function of my DatePicker. My code works well, but if I select 2 employees that CAN go on vacation the same date, it will also display the error message. I have also tried my If statement with And instead of Or, but it keeps displaying the error message when it doesn't have to.

Collect(MyVacation,VacationRequest);
If(Or(Value(txtEmpID.Text) = 4 && (CountRows(Filter(MyVacation, EmployeeID = 3))>0 && (CountRows(Filter(MyVacation, StartDate = FilterDatePicker.SelectedDate))>0))), Notify("Someone in your department has already choose this date for leave. Please select an other date", NotificationType.Error, 4000),
(Value(txtEmpID.Text) = 5 && (CountRows(Filter(MyVacation, EmployeeID = 6))>0 && (CountRows(Filter(MyVacation, StartDate = FilterDatePicker.SelectedDate))>0))), Notify("Someone in your department has already choose this date for leave. Please select an other date", NotificationType.Error,4000),
(Value(txtEmpID.Text) = 6 && (CountRows(Filter(MyVacation, EmployeeID = 5))>0 && (CountRows(Filter(MyVacation, StartDate = FilterDatePicker.SelectedDate))>0))), Notify("Someone in your department has already choose this date for leave. Please select an other date", NotificationType.Error,4000))

 

Can someone help me with my code?

1 ACCEPTED SOLUTION

Accepted Solutions

Thanks @JB0007 ,

It is always better spending my time analyzing code logic rather than trying to work out all the brackets and commas.

So, firstly, you have some brackets you do not need (although they will not affect the formula), but I see a fundamental flaw in your logic as the CountRows() are independent of each other - they are simply saying that (for the first one) if EmployeeID 4 is in the text box, then EmployeeID 3 might have some vacation somewhere in the list AND someone might have vacation on that day.

ClearCollect(
    MyVacation,
    VacationRequest
);
If(
   Value(txtEmpID.Text) = 4 &&
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 3
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);
If(
   Value(txtEmpID.Text) = 5 && 
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 6
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);
If(
   Value(txtEmpID.Text) = 6 && 
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 5
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);

I will do the first one here

If(
   Value(txtEmpID.Text) = 4 &&
   !IsBlank(
      Lookup(
         MyVacation,
         EmployeeID = 3 && 
         StartDate = FilterDatePicker.SelectedDate,
         EmployeeID
      )
   ),
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);

I am still a little confused over your logic with the EmployeeID, however the code above should return the result you require based on what you have input.

 

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

 

View solution in original post

7 REPLIES 7
WarrenBelz
Super User
Super User

Hi @JB0007 ,

Can you please use Format Text and post this in a code box - </> icon above

Hi @WarrenBelz , thank you so much for the reply. I have formatted the code as you requested.

<Collect(
       MyVacation,
       VacationRequest
);

If(
      Value(txtEmpID.Text) = 4 && (CountRows(
      Filter(
         MyVacation,
         EmployeeID = 3
      )
      ) > 0 && (CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
      ) > 0)),
      Notify(
         "Someone in your department has already choose this date for leave. Please select an other date",
         NotificationType.Error,
         4000
)
);
If(
      Value(txtEmpID.Text) = 5 && (CountRows(
      Filter(
         MyVacation,
         EmployeeID = 6
      )
      ) > 0 && (CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
      ) > 0)),
      Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
)
);
If(
      Value(txtEmpID.Text) = 6 && (CountRows(
      Filter(
         MyVacation,
         EmployeeID = 5
      )
      ) > 0 && (CountRows(
    Filter(
        MyVacation,
        StartDate = FilterDatePicker.SelectedDate
    )
    ) > 0)),
    Notify(
    "Someone in your department has already choose this date for leave. Please select an other date",
     NotificationType.Error,
     4000
)
);>

@JB0007 ,

Can you please put this in a code box so all the indents work - it is the </> icon above.

Also you have not used Format Text as there are still brackets on the right of code?

Collect(
    MyVacation,
    VacationRequest
);
If(
    Value(txtEmpID.Text) = 4 && (CountRows(
        Filter(
            MyVacation,
            EmployeeID = 3
        )
    ) > 0 && (CountRows(
        Filter(
            MyVacation,
            StartDate = FilterDatePicker.SelectedDate
        )
    ) > 0)),
    Notify(
        "Someone in your department has already choose this date for leave. Please select an other date",
        NotificationType.Error,
        4000
    )
);
If(
    Value(txtEmpID.Text) = 5 && (CountRows(
        Filter(
            MyVacation,
            EmployeeID = 6
        )
    ) > 0 && (CountRows(
        Filter(
            MyVacation,
            StartDate = FilterDatePicker.SelectedDate
        )
    ) > 0)),
    Notify(
        "Someone in your department has already choose this date for leave. Please select an other date",
        NotificationType.Error,
        4000
    )
);
If(
    Value(txtEmpID.Text) = 6 && (CountRows(
        Filter(
            MyVacation,
            EmployeeID = 5
        )
    ) > 0 && (CountRows(
        Filter(
            MyVacation,
            StartDate = FilterDatePicker.SelectedDate
        )
    ) > 0)),
    Notify(
        "Someone in your department has already choose this date for leave. Please select an other date",
        NotificationType.Error,
        4000
    )
);


Here you go @WarrenBelz , I didn't understand the first time (I'm new into Power Apps).

Thanks @JB0007 ,

It is always better spending my time analyzing code logic rather than trying to work out all the brackets and commas.

So, firstly, you have some brackets you do not need (although they will not affect the formula), but I see a fundamental flaw in your logic as the CountRows() are independent of each other - they are simply saying that (for the first one) if EmployeeID 4 is in the text box, then EmployeeID 3 might have some vacation somewhere in the list AND someone might have vacation on that day.

ClearCollect(
    MyVacation,
    VacationRequest
);
If(
   Value(txtEmpID.Text) = 4 &&
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 3
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);
If(
   Value(txtEmpID.Text) = 5 && 
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 6
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);
If(
   Value(txtEmpID.Text) = 6 && 
   CountRows(
      Filter(
         MyVacation,
         EmployeeID = 5
      )
   ) > 0 && 
   CountRows(
      Filter(
         MyVacation,
         StartDate = FilterDatePicker.SelectedDate
      )
   ) > 0,
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);

I will do the first one here

If(
   Value(txtEmpID.Text) = 4 &&
   !IsBlank(
      Lookup(
         MyVacation,
         EmployeeID = 3 && 
         StartDate = FilterDatePicker.SelectedDate,
         EmployeeID
      )
   ),
   Notify(
      "Someone in your department has already choose this date for leave. Please select an other date",
      NotificationType.Error,
      4000
   )
);

I am still a little confused over your logic with the EmployeeID, however the code above should return the result you require based on what you have input.

 

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

 

Hi @WarrenBelz, I have tried your code and it works perfectly. I have used the EmployeeID because:
1. It is the value that I wanted the use to compare and look for other value.

2. It is easier than using for e.x. EmployeeName because I can type the name with an error.

 

Further, I want to ask you why did you put another EmployeeID after FilterDatepicker.SelectedDate?

If(
   Value(txtEmpID.Text) = 4 &&
   !IsBlank(
      Lookup(
         MyVacation,
         EmployeeID = 3 && 
         StartDate = FilterDatePicker.SelectedDate,
         EmployeeID
      )
   ),

 

Your answer could help me better understand what I am going to do next time for a similar project.


 

Hi @JB0007 ,

That can actually be any field in the target data set (I just knew Employee ID would be there) - if in doubt, use ID.

It generally works without it, but the Lookup is then returning all fields in the first matching record, rather than a single field.

You would use the syntax without this to populate a form with a given record.

Helpful resources

Announcements

Super User of the Month | Drew Poggemann

As part of a new monthly feature in the Community, we are excited to share that Drew Poggemann is our featured Super User for the month of February 2024. If you've been in the Community for a while, we're sure Drew's name is familiar to you, as he is one of our most active contributors--he's been a Super User for five consecutive seasons!   Since authoring his first reply 5 years ago to his 514th solution authored, Drew has helped countless Community members with his insights and expertise. In addition to being a Super User, Drew is also a User Group leader and a Microsoft MVP. His contributions to our Super User sessions and to the new SUIT program are always welcome--as well as his sense of humor and fun-loving way of sharing what he knows with others.   When Drew is not solving problems and authoring solutions, he's busy overseeing the Solution Architecture team at HBS, specializing in application architecture and business solution strategy--something he's been doing for over 30 years. We are grateful for Drew and the amazing way he has used his talent and skills to help so many others in the Community. If you are part of the SUIT program, you got to hear some great tips from Drew at the first SUIT session--and we know he still has much more to share!You can find him in the Community and on LinkedIn. Thank you for all you do, Drew!

Announcing Power Apps Copilot Cookbook Gallery

We are excited to share that the all-new Copilot Cookbook Gallery for Power Apps is now available in the Power Apps Community, full of tips and tricks on how to best use Microsoft Copilot as you develop and create in Power Apps. The new Copilot Cookbook is your go-to resource when you need inspiration--or when you're stuck--and aren't sure how to best partner with Copilot while creating apps.   Whether you're looking for the best prompts or just want to know about responsible AI use, visit Copilot Cookbook for regular updates you can rely on--while also serving up some of your greatest tips and tricks for the Community. Our team will be reviewing posts using the new "Copilot Studio" label to ensure we highlight and amplify the most relevant and recent content, so you're assured of high-quality content every time you visit. If you share a post that gets featured in the curated gallery, you'll get a PM in the Community to let you know!The curated gallery is ready for you to experience now, so visit the new Copilot Cookbook for Power Apps today: Copilot Cookbook - Power Platform Community. We can't wait to see what you "cook" up!      

Celebrating a New Season of Super Users with Charles Lamanna, CVP Microsoft Business Applications

February 8 was the kickoff to the 2024 Season One Super User program for Power Platform Communities, and we are thrilled to welcome back so many returning Super Users--as well as so many brand new Super Users who started their journey last fall. Our Community Super Users are the true heroes, answering questions, providing solutions, filtering spam, and so much more. The impact they make on the Communities each day is significant, and we wanted to do something special to welcome them at our first kickoff meeting of the year.   Charles Lamanna, Microsoft CVP of Business Applications, has stressed frequently how valuable our Community is to the growth and potential of Power Platform, and we are honored to share this message from him to our 2024 Season One Super Users--as well as anyone who might be interested in joining this elite group of Community members.     If you want to know more about Super Users, check out these posts for more information today:    Power Apps: What is A Super User? - Power Platform CommunityPower Automate: What is A Super User? - Power Platform Community Copilot Studio: What is A Super User? - Power Platform Community Power Pages: What is A Super User? - Power Platform Community

February 2024 User Group Update: Welcoming New Groups and Highlighting Upcoming Events

It's a new month and a brand-new year, which means another opportunity to celebrate our amazing User Groups!Each month, we highlight the new User Groups that have joined the community. It's been a busy season for new groups, because we are thrilled to welcome 15 New User Groups! Take a look at the list below, shared by the different community categories. If your group is listed here, give this post a kudo so we can celebrate with you!   We love our User Groups and the difference they make in the lives of our Community! Thank you to all the new User Groups, new User Group leaders--we look forward to hearing about your successes and the impact you will leave!   In addition to our monthly New User Group spotlight, it's a great time to share some of the latest events happening in our User Group community! Take a look at the list below to find one that fits your schedule and need! There's a great combination of in-person and virtual events to choose from. It's a great time of year to connect and engage with User Groups both locally and online. Please Welcome Our NEW User Groups   Power Platform: Heathcare Power Platform User Group Power Platform Connect Hub Power Platform Usergroup Denmark Mexico Norte- Power Platform User Group Pune Power User Group Sudbury Power Platform User GroupMicrosoft User Group GhanaMPPBLR - Microsoft Power Platform Bengaluru User Group Power Apps:   Myrtle Beach Power Platform User GroupAnanseTechWB PowerApps Copilot Studio: Pathfinders Power Platform Community Dynamics365: Cairo, Egypt MSD 365 Business Central/NAV/F&O User GruopMS Dynamics 365 Business Central LatamCincinnati OH D365 F&O User Group February User Group Events February 2024 Cleveland Power Platform User GroupPortallunsj - Februar 2024Indiana D365/AX February User Group MeetingQ1 2024 KC Power Platform and Dynamics 365 CRM Users Group 

Super Users 2024 Season One is Here!

   We are excited to announce the first season of our 2024 Super Users is here! Our kickoff to the new year welcomes many returning Super Users and several new faces, and it's always exciting to see the impact these incredible individuals will have on the Community in 2024! We are so grateful for the daily difference they make in the Community already and know they will keep staying engaged and excited for all that will happen this year.   How to Spot a Super User in the Community:Have you ever written a post or asked for help in the Community and had it answered by a user with the Super User icon next to their name? It means you have found the actual, real-life superheroes of the Power Platform Community! Super Users are our heroes because of the way they consistently make a difference in the Community. Our amazing Super Users help keep the Community a safe place by flagging spam and letting the Community Managers know about issues. They also make the Community a great place to find answers, because they are often the first to offer solutions and get clarity on questions. Finally, Super Users share valuable insights on ways to keep the Community growing, engaging, and looking ahead!We are honored to reveal the new badges for this season of Super Users! Congratulations to all the new and returning Super Users!     To better answer the question "What is a Super User?" please check out this article: Power Apps: What is A Super User? - Power Platform CommunityPower Automate: What is A Super User? - Power Platform Community Copilot Studio: What is A Super User? - Power Platform Community Power Pages: What is A Super User? - Power Platform Community

Did You Attend the Microsoft Power Platform Conference in 2022 or 2023? Claim Your Badge Today!

If you were one of the thousands of people who joined us at the first #MPPC Microsoft Power Platform Conference in 2022 in Orlando--or attended the second-annual conference in Las Vegas in 2023--we are excited to honor you with a special community badge! Show your support for #MPPC Microsoft Power Platform Conference this year by claiming your badge!         Just follow this link to claim your badge for attending #MPPC in 2022 and/or 2023: MPPCBadgeRequest    Want to earn your badge for 2024? Just keep watching our News & Announcements for the latest updates on #MPPC24.

Top Solution Authors
Top Kudoed Authors
Users online (2,920)