Uploaded image for project: 'Apache Cordova'
  1. Apache Cordova
  2. CB-2502

iOS: CDVViewController's commandDelegate property should be typed to the protocol

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • Master
    • 2.6.0
    • cordova-ios
    • None

    Description

      CDVViewController's commandDelegate property is typed as CDVCommandDelegateImpl, the concrete implementation of the CDVCommandDelegate protocol. From an object-oriented design standpoint, we want to property to conform to the protocol API, not the implementation. This would come into play particularly if a consumer was providing their own implementation of the protocol, as an override to the parent class's definition.

      Attachments

        Activity

          People

            agrieve Andrew Grieve
            khawkins@salesforce.com Kevin Hawkins
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 2h
                2h
                Remaining:
                Remaining Estimate - 2h
                2h
                Logged:
                Time Spent - Not Specified
                Not Specified