Skip to main content

You are not logged in. Your edit will be placed in a queue until it is peer reviewed.

We welcome edits that make the post easier to understand and more valuable for readers. Because community members review edits, please try to make the post substantially better than how you found it, for example, by fixing grammar or adding additional resources and hyperlinks.

Required fields*

2
  • 5
    You have basically replaced IsFreeVersion with FeaturexEnabled, you haven't reduced the number of calls. While I haven't had exactly that case I've always handled similar stuff by upon menu creation disabling those options that the user shouldn't see. Mostly this is at form creation but I've sometimes had to do it when preparing a pop-up menu. Commented May 16, 2018 at 3:17
  • 1
    @LorenPechtel: you should read my answer again, more carefully. I actually mentioned two things to reduce the number of conditional tests, one of them the DRY principle, one of them focussing on tests in UI. More important, mapping an unspecific flag like isFreeVersionto specific feature parameters removes most of the pain of those tests - they will actually start to make sense and don't produce a maintenance mess any more. Commented May 16, 2018 at 5:48