Uploaded image for project: 'In-Portal CMS'
  1. In-Portal CMS
  2. INP-366

Converting "Unit Config" files from Arrays to Classes

    XMLWordPrintable

    Details

    • Type: Feature Request
    • Status: Closed
    • Priority: Minor
    • Resolution: no change required
    • Affects Version/s: 5.1.0
    • Fix Version/s: 5.3.0-B1
    • Component/s: Optimization
    • Labels:
      None

      Description

      Goal is to convert all unit configs from array to classes form.

      This will give us such immediate benefits:

      • all common unit config settings, that are copied from one unit config to another will be located in parent class (saves space);
      • all unit config setting names will be automatically suggested by Zend, and this will eliminate most of setting name typing errors;
      • we will have a list of all possible unit config settings with their description, purpose and usage examples.

      We should create base class for unit configs where all mentioned above. I think, that all unit configs (system and user-made) should be named the same way and need to use some prefix (along with suffix as usual), that will prevent them from appearing in class-name auto-complete dialogs at least user really wants them to be there.

      Unit config class name could be formed by coding standards using prefix-name, declared in this config. For example "ucUserConfig" (for prefix
      "user"), "ucAgentConfig" (for prefix "agent"), "ucCustomSection" (for prefix "custom-section") and so on.

      [B]We need to do TESTs before converting all Unit Configs:[/B]

      1. Performance
      2. Memory Usage
      3. Code Usability

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                alex Alex
                Reporter:
                alex Alex
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: