Uploaded image for project: 'FOP'
  1. FOP
  2. FOP-2785

inconsistent method name "make"

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Duplicate
    • 2.5
    • None
    • None
    • None

    Description

      The method seems to find an appropriate "Property" related to the fo.getNameId().
      Thus, identifier "find" should be better than "make".

         /**
           * Set default precedence according to the parent FObj
           *
           * {@inheritDoc}
           */
           public Property make(PropertyList propertyList) throws PropertyException {
               FObj fo = propertyList.getFObj();
               switch (fo.getNameId()) {
               case Constants.FO_TABLE:
                   return num6;
               case Constants.FO_TABLE_CELL:
                   return num5;
               case Constants.FO_TABLE_COLUMN:
                   return num4;
               case Constants.FO_TABLE_ROW:
                   return num3;
               case Constants.FO_TABLE_BODY:
                   return num2;
               case Constants.FO_TABLE_HEADER:
                   return num1;
               case Constants.FO_TABLE_FOOTER:
                   return num0;
               default:
                   return null;
               }
           }
      

      Attachments

        1. rename-method.patch
          1 kB
          KuiLIU

        Activity

          People

            Unassigned Unassigned
            BruceKuiLIU KuiLIU
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: