Non-core contributed modules

Themed Resource

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Incomplete
  • Affects Version/s: 1.9
  • Fix Version/s: None
  • Component/s: Add a project here
  • Labels:
    None

Description

Moodlerooms Inc, would like to contribute the themed resuorce back to the Moodle community and have it added to the contrib CVS.

Project Description

The theme resource was designed and developed to provide an instructor with a way to add multiple Moodle HTML resources to multiple courses with the same look and feel without a lot of inline CSS styles. The themed resource allows an instructor to design an html template outside of Moodle using HTML and CSS and upload this template to the Moodle themed resource and user it over and over again with custom fields. Once a template has been added to a Moodle course it can be reused in any themed resource in that course. The resources and be backed up and restore to another course and that course can use the same html template. The themed resource provides instructors with a quick way to create many html resources with the same look and feel without a lot of copying and pasting of the html structure.

Developers

Sam Chaffee and Jason Hardin

Activity

Hide
Anthony Borrow added a comment -

Jason - Sorry for the delay in getting this uploaded. A couple of comment. First, I think it is important because of the security implications to put that in BOLD and mention it write off in the M&P database. The zip file you attached seems to have the code duplicated, if I understand the code correctly. You seem to have an extra themed_resource directory. I'm also curious why the lang file could not go within the resource type and use get_string('resourcetypethemed','resource_themed'). I've not used (or seen) the _local extension added at the end of en_utf8_local. I think you could get everything under /mod/resource/lang/en_utf8/themed.php and keep it all in one directory. Let me know what you think and we can move forward from there. Peace - Anthony

Show
Anthony Borrow added a comment - Jason - Sorry for the delay in getting this uploaded. A couple of comment. First, I think it is important because of the security implications to put that in BOLD and mention it write off in the M&P database. The zip file you attached seems to have the code duplicated, if I understand the code correctly. You seem to have an extra themed_resource directory. I'm also curious why the lang file could not go within the resource type and use get_string('resourcetypethemed','resource_themed'). I've not used (or seen) the _local extension added at the end of en_utf8_local. I think you could get everything under /mod/resource/lang/en_utf8/themed.php and keep it all in one directory. Let me know what you think and we can move forward from there. Peace - Anthony
Hide
Jason Hardin added a comment -

Anthony,
Both valid points. I will add the statement to the Modules and Plug ins database.

I will need to look at the zip again I didn't realize there were two directories for the themed resource and there should not have been an en_utf8_local directory in the zip.

Show
Jason Hardin added a comment - Anthony, Both valid points. I will add the statement to the Modules and Plug ins database. I will need to look at the zip again I didn't realize there were two directories for the themed resource and there should not have been an en_utf8_local directory in the zip.
Hide
Anthony Borrow added a comment -

Jason - Thanks for looking at these issues. Just update the zip file and then I'll re-evaluate it for CONTRIB. Peace - Anthony

Show
Anthony Borrow added a comment - Jason - Thanks for looking at these issues. Just update the zip file and then I'll re-evaluate it for CONTRIB. Peace - Anthony
Hide
Anthony Borrow added a comment -

I never heard back with an updated file. I am going to go ahead and close this issue as won't fix or incomplete. If you want it added to CONTRIB just let me know and I'll review it again. Sorry for dropping the ball. Now that 2.0 is out I hope to pick up a few things that I had fallen behind on. Peace - Anthony

Show
Anthony Borrow added a comment - I never heard back with an updated file. I am going to go ahead and close this issue as won't fix or incomplete. If you want it added to CONTRIB just let me know and I'll review it again. Sorry for dropping the ball. Now that 2.0 is out I hope to pick up a few things that I had fallen behind on. Peace - Anthony
Hide
Anthony Borrow added a comment -

If I hear back that there is a revised version then we can update. Otherwise, if it does not seem needed then I will just ignore this request. Peace - Anthony

Show
Anthony Borrow added a comment - If I hear back that there is a revised version then we can update. Otherwise, if it does not seem needed then I will just ignore this request. Peace - Anthony
Hide
Anthony Borrow added a comment -

Jason - The M&P entry is at http://moodle.org/mod/data/view.php?d=13&rid=1828&filter=1 and points to the MoodleRooms development server. This seems to be functional and at this point may not be worth the trouble of adding to CONTRIB. Either way is fine with me. Peace - Anthony

Show
Anthony Borrow added a comment - Jason - The M&P entry is at http://moodle.org/mod/data/view.php?d=13&rid=1828&filter=1 and points to the MoodleRooms development server. This seems to be functional and at this point may not be worth the trouble of adding to CONTRIB. Either way is fine with me. Peace - Anthony

People

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: