CRM-16112 Ensure type is set on all fields in api getfields

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Trivial
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.6
          • Fix Version/s: 4.6
          • Component/s: None
          • Labels:
            None
          • Documentation Required?:
            None
          • Funding Source:
            Contributed Code

            Description

            Add a syntax conformance test to ensure type is always set for getfields & fix existing instances.

            I've been hitting e-notices relating to this and it seems that consistent declaration is more useful than handling e-notices downstream

              Attachments

                Activity

                [CRM-16112] Ensure type is set on all fields in api getfields
                Eileen McNaughton added a comment -

                https://github.com/civicrm/civicrm-core/pull/5395

                John K. added a comment -

                The above PR was rebased here https://github.com/civicrm/civicrm-core/pull/5397 and is now showing as merged, so I believe this issue is fixed/closed?

                Eileen McNaughton added a comment -

                Yep close enough - we didn't get to the bottom of the type 'array' - which is an oddity. But good enough for now

                  People

                  • Assignee:
                    Eileen McNaughton
                    Reporter:
                    Eileen McNaughton

                    Dates

                    • Created:
                      Updated:
                      Resolved: