Uploaded image for project: 'COmanage'
  1. COmanage
  2. CO-368

validateExtendedType Should Use Dynamic Validation Rules

    XMLWordPrintable

Details

    Description

      AppModel::validateExtendedType needs to know the current CO ID in order to figure out which types are valid. However, $cur_co is attached to the Controller, not the Model.

      The correct way to handle this would be to use CakePHP 2.2's dynamic validation rules

      http://book.cakephp.org/2.0/en/models/data-validation.html#dynamically-change-validation-rules

      to attach a suitable rule to the model, possibly in AppController. Currently, AppController::beforeFilter explicitly passes the CO ID to the Identifier model (the only model that currently supports Extended Types).

      Attachments

        Issue Links

          depends on

          Task - A task that needs to be done. CO-369 Update to CakePHP 2.3

          • Blocker - Blocks development and/or testing work, production could not run.
          • Resolved
          has dependent

          New Feature - A new feature of the product, which has yet to be developed. CO-370 Expand Extended Types to More Models

          • Major - Major loss of function.
          • Resolved
          is related to

          New Feature - A new feature of the product, which has yet to be developed. CO-362 Allow Custom Type Definitions

          • Major - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. CO-413 Petition Creation Fails on Identifier

          • Blocker - Blocks development and/or testing work, production could not run.
          • Resolved

          Activity

            People

              benn.oshrin@at.internet2.edu Benn Oshrin (internet2.edu)
              benn.oshrin@at.internet2.edu Benn Oshrin (internet2.edu)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: