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

allow cloning of core permissions in plugins

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1, 2.2
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Roles / Access
    • Labels:

      Description

      at present you can clone in core or in one plugin only, for upgrades of addinstance cap we need to clone from core-->module

      and update: http://docs.moodle.org/dev/Access_API#How_to_define_new_capabilities_in_plugins

        Gliffy Diagrams

          Activity

          Hide
          poltawski Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          poltawski Dan Poltawski added a comment -

          Hi Petr,

          Do you think this is necessary to put on the stable branches?

          Show
          poltawski Dan Poltawski added a comment - Hi Petr, Do you think this is necessary to put on the stable branches?
          Hide
          skodak Petr Skoda added a comment -

          The problem with stable is that ppl often install newer contrib versions in older moodle versions - in general that is not discouraged and should work, but in case of new addinstance capability they would loose the migration of custom roles if this issue was not backported.

          Show
          skodak Petr Skoda added a comment - The problem with stable is that ppl often install newer contrib versions in older moodle versions - in general that is not discouraged and should work, but in case of new addinstance capability they would loose the migration of custom roles if this issue was not backported.
          Hide
          poltawski Dan Poltawski added a comment -

          Hmm, for some reason this test seems to run very slow for me

          Show
          poltawski Dan Poltawski added a comment - Hmm, for some reason this test seems to run very slow for me
          Hide
          poltawski Dan Poltawski added a comment -

          Thanks, this has been integrated now.

          Show
          poltawski Dan Poltawski added a comment - Thanks, this has been integrated now.
          Hide
          salvetore Michael de Raadt added a comment -

          I'm going to run the phpunit tests in a couple of other issues. I can let you know how that goes.

          Show
          salvetore Michael de Raadt added a comment - I'm going to run the phpunit tests in a couple of other issues. I can let you know how that goes.
          Hide
          abgreeve Adrian Greeve added a comment -

          I ran the phpunit tests in master and then tested installing ouwiki in 2.1 and 2.2. Besides the module missing a whole bunch of language strings, everything else worked well.
          Thanks.

          Show
          abgreeve Adrian Greeve added a comment - I ran the phpunit tests in master and then tested installing ouwiki in 2.1 and 2.2. Besides the module missing a whole bunch of language strings, everything else worked well. Thanks.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          This has been near becoming rejected, because it's not the best code you are able to produce.

          But, luckily, at the end, it has landed and has been spread to all repos out there.

          Many thanks and, don't forget it, keep improving your skills, you can!

          Closing, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - This has been near becoming rejected, because it's not the best code you are able to produce. But, luckily, at the end, it has landed and has been spread to all repos out there. Many thanks and, don't forget it, keep improving your skills, you can! Closing, ciao

            People

            • Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/May/12