Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-22317

page and resource modules... revision field incremented but not used...

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Resource
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      While implementing page and resource modules I've found that it also has the same "revision" field than imscp (MDL-22315), but it seems that it isn't it use at all.

      Am i wrong? Are versions stored ?

      My +1 to drop that both from here and from imscp.

      Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            skodak Petr Skoda added a comment -

            the revision field is necessary to prevent caching problems, if it is not used it is a bug which needs to be fixed, I thought I tested it, but it is a long time ago, I am going to retest everything in these modules and update them a bit, thanks

            Show
            skodak Petr Skoda added a comment - the revision field is necessary to prevent caching problems, if it is not used it is a bug which needs to be fixed, I thought I tested it, but it is a long time ago, I am going to retest everything in these modules and update them a bit, thanks
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Ah,

            I told it was not being used... from a File Storage point of view (I did missed the "caching" point completely).

            So my suggestion to drop it was caused because of the "abuse" (IMO) we are doing in the IMSCP module of the files->itemid column for storing multiple revisions for... nothing?

            So, it revision is being used to solve caching, np at all. Consider this not a bug. About the imscp problem, MDL-22315 in the one to be fixed.

            Thanks and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Ah, I told it was not being used... from a File Storage point of view (I did missed the "caching" point completely). So my suggestion to drop it was caused because of the "abuse" (IMO) we are doing in the IMSCP module of the files->itemid column for storing multiple revisions for... nothing? So, it revision is being used to solve caching, np at all. Consider this not a bug. About the imscp problem, MDL-22315 in the one to be fixed. Thanks and ciao
            Hide
            skodak Petr Skoda added a comment -

            not a bug, you can restore anything into page and resource module

            Show
            skodak Petr Skoda added a comment - not a bug, you can restore anything into page and resource module

              People

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

                Dates

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