Details

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

      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

        Activity

        Hide
        Dongsheng Cai added a comment -

        HI Petr

        what is your suggestion to fix this?

        Show
        Dongsheng Cai added a comment - HI Petr what is your suggestion to fix this?
        Hide
        Petr Škoda 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
        Petr Škoda 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 Cai added a comment -

        Use static variable instead

        Show
        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: