Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Comments
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      $CFG->commentcommentcode should not be in config table - we are trying hard to get rid of all these, please do not add more runtime rubbish to $CFG

        Gliffy Diagrams

          Activity

          Hide
          dongsheng Dongsheng Cai added a comment -

          HI Petr

          what is your suggestion to fix this?

          Show
          dongsheng Dongsheng Cai added a comment - HI Petr what is your suggestion to fix this?
          Hide
          skodak Petr Skoda added a comment -

          hmm, it could be done as a comments renderer property, or at present as a static variable in function (this is hard to unit test though)

          Show
          skodak Petr Skoda added a comment - hmm, it could be done as a comments renderer property, or at present as a static variable in function (this is hard to unit test though)
          Hide
          dongsheng Dongsheng Cai added a comment -

          Use static variable instead

          Show
          dongsheng Dongsheng Cai added a comment - Use static variable instead

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                24/Nov/10