Non-core contributed modules

Request booking move form

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: None
  • Component/s: Block: Mrbs
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE

Description

If a user viewing a booking would be able to book the room (if the booking weren't there) this patch prints a form to send the creator of the booking a pre-generated message via the standard moodle messaging system.

In it's current state it needs a small patch to message/send.php (a single line), but it might be able to be worked such that this isn't required.

TODO:
Make pre-generated message configurable (maybe use langfiles for default)
Put Request that this booking be moved in langfile

  1. messagereferdiff.txt
    22/Apr/09 8:03 PM
    0.2 kB
    Mike Worth
  2. mrbsrequestdiff.txt
    20/May/09 10:22 PM
    2 kB
    Mike Worth
  3. mrbsrequestdiff.txt
    22/Apr/09 8:03 PM
    2 kB
    Mike Worth

Activity

Hide
Anthony Borrow added a comment -

Mike - As I understand this, it would be a nice way of asking someone if they could move their booking to make room for someone else? Let's try to avoid the patch to Moodle core since the MRBS block is technically a plugin. I'll look and see what the change is and how we might get around it. Peace- Anthony

Show
Anthony Borrow added a comment - Mike - As I understand this, it would be a nice way of asking someone if they could move their booking to make room for someone else? Let's try to avoid the patch to Moodle core since the MRBS block is technically a plugin. I'll look and see what the change is and how we might get around it. Peace- Anthony
Hide
Anthony Borrow added a comment -

Mike - What file gets modified, the diff file you uploaded for messagereferer did not seem to indicate the file. It just showed the line being adding (not the one being removed). Peace - Anthony

Show
Anthony Borrow added a comment - Mike - What file gets modified, the diff file you uploaded for messagereferer did not seem to indicate the file. It just showed the line being adding (not the one being removed). Peace - Anthony
Hide
Mike Worth added a comment -

message/send.php

I think I might make it such that it opens the message popup, which the user will have to then close.

Show
Mike Worth added a comment - message/send.php I think I might make it such that it opens the message popup, which the user will have to then close.
Hide
Mike Worth added a comment -

Now opens send.php in new window; removed need for patch to core code.

Show
Mike Worth added a comment - Now opens send.php in new window; removed need for patch to core code.
Hide
Anthony Borrow added a comment -

Nice, glad to see you were able to avoid modifying core. Peace - Anthony

Show
Anthony Borrow added a comment - Nice, glad to see you were able to avoid modifying core. Peace - Anthony
Hide
Mike Worth added a comment -

Committed to CVS, didn't bother with making the message configurable (just put it in langfile)- it should be generic enough, if not then users will have to use local lang strings

Show
Mike Worth added a comment - Committed to CVS, didn't bother with making the message configurable (just put it in langfile)- it should be generic enough, if not then users will have to use local lang strings
Hide
Anthony Borrow added a comment -

Mike - Thanks for the commit. No need to make it configurable with the way language strings are handled. As you note, a site admin can always update their own local language pack as needed. Peace - Anthony

Show
Anthony Borrow added a comment - Mike - Thanks for the commit. No need to make it configurable with the way language strings are handled. As you note, a site admin can always update their own local language pack as needed. Peace - Anthony

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: