core MDL-19799 Added set_url calls and replaced deprecated functions

safatshahin · Oct 15, 2009 · 474f6bf · 474f6bf
1 parent ea6c02f
commit 474f6bf
Show file tree

Hide file tree

Showing 7 changed files with 36 additions and 10 deletions.
diff --git a/help.php b/help.php
@@ -23,6 +23,27 @@
 
 $PAGE->set_course($COURSE);
 
+$url = new moodle_url($CFG->wwwroot.'/help.php');
+if ($file !== '')  {
+    $url->param('file', $file);
+}
+if ($text !== 'No text to display')  {
+    $url->param('text', $text);
+}
+if ($module !== 'moodle')  {
+    $url->param('module', $module);
+}
+if ($forcelang !== '')  {
+    $url->param('forcelang', $forcelang);
+}
+if ($skiplocal !== 0)  {
+    $url->param('skiplocal', $skiplocal);
+}
+if ($fortooltip !== 0)  {
+    $url->param('fortooltip', $fortooltip);
+}
+$PAGE->set_url($url);
+
 // We look for the help to display in lots of different places, and
 // only display an error at the end if we can't find the help file
 // anywhere. This variable tracks that.
@@ -93,6 +114,7 @@
 }
 
 // Do the main output.
+$PAGE->set_generaltype('popup');
 $PAGE->set_title($title);
 echo $OUTPUT->header();
 echo $OUTPUT->box_start();

diff --git a/index.php b/index.php
@@ -1,4 +1,4 @@
-<?php  // $Id$
+<?php
        // index.php - the front page.
 
 ///////////////////////////////////////////////////////////////////////////
@@ -138,7 +138,7 @@
         }
     }
 
-    if (isloggedin() and !isguest() and isset($CFG->frontpageloggedin)) {
+    if (isloggedin() and !isguestuser() and isset($CFG->frontpageloggedin)) {
         $frontpagelayout = $CFG->frontpageloggedin;
     } else {
         $frontpagelayout = $CFG->frontpage;
@@ -176,10 +176,10 @@
 
             case FRONTPAGECOURSELIST:
 
-                if (isloggedin() and !has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)) and !isguest() and empty($CFG->disablemycourses)) {
+                if (isloggedin() and !has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)) and !isguestuser() and empty($CFG->disablemycourses)) {
                     echo $OUTPUT->heading(get_string('mycourses'), 2, 'headingblock header');
                     print_my_moodle();
-                } else if ((!has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)) and !isguest()) or ($DB->count_records('course') <= FRONTPAGECOURSELIMIT)) {
+                } else if ((!has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM)) and !isguestuser()) or ($DB->count_records('course') <= FRONTPAGECOURSELIMIT)) {
                     // admin should not see list of courses when there are too many of them
                     echo $OUTPUT->heading(get_string('availablecourses'), 2, 'headingblock header');
                     print_courses(0);

diff --git a/lib/javascript-static.js b/lib/javascript-static.js
@@ -830,7 +830,7 @@ function openpopup(event, args) {
     if (!windowobj) {
         return true;
     }
-    if (fullscreen) {
+    if (args.fullscreen) {
         windowobj.moveTo(0,0);
         windowobj.resizeTo(screen.availWidth,screen.availHeight);
     }

diff --git a/user/index.php b/user/index.php
@@ -771,7 +771,7 @@
                     if (!empty($user->role)) {
                         $row->cells[1]->text .= get_string('role') .': '. $user->role .'<br />';
                     }
-                    if ($user->maildisplay == 1 or ($user->maildisplay == 2 and ($course->id != SITEID) and !isguest()) or
+                    if ($user->maildisplay == 1 or ($user->maildisplay == 2 and ($course->id != SITEID) and !isguestuser()) or
                                 has_capability('moodle/course:viewhiddenuserfields', $context)) {
                         $link = new html_link();
                         $link->url = "mailto:$user->email";

diff --git a/user/view.php b/user/view.php
@@ -289,7 +289,7 @@
 }
 
 if ($user->maildisplay == 1 or
-   ($user->maildisplay == 2 and ($course->id != SITEID) and !isguest()) or
+   ($user->maildisplay == 2 and ($course->id != SITEID) and !isguestuser()) or
    has_capability('moodle/course:useremail', $coursecontext)) {
 
     $emailswitch = '';
@@ -536,7 +536,7 @@
     echo '</form>';
 }
 
-if (!$user->deleted and !empty($CFG->messaging) and !isguest() and has_capability('moodle/site:sendmessage', $systemcontext)) {
+if (!$user->deleted and !empty($CFG->messaging) and !isguestuser() and has_capability('moodle/site:sendmessage', $systemcontext)) {
     if (!empty($USER->id) and ($USER->id == $user->id)) {
         if ($countmessages = $DB->count_records('message', array('useridto'=>$user->id))) {
             $messagebuttonname = get_string("messages", "message")."($countmessages)";

diff --git a/userpix/index.php b/userpix/index.php
@@ -1,4 +1,4 @@
-<?php // $Id$
+<?php
       // This simple script displays all the users with pictures on one page.
       // By default it is not linked anywhere on the site.  If you want to
       // make it available you should link it in yourself from somewhere.
@@ -8,6 +8,8 @@
 
     include('../config.php');
 
+    $PAGE->set_url(new moodle_url($CFG->wwwroot.'/userpix/index.php'));
+
     require_login();
 
 /// Remove the following three lines if you want everyone to access it

diff --git a/userpix/upgrade.php b/userpix/upgrade.php
@@ -1,10 +1,12 @@
-<?php // $Id$
+<?php
       // This script updates all users picturesi to remove black border.
 
 
     include('../config.php');
     include('../lib/gdlib.php');
 
+    $PAGE->set_url(new moodle_url($CFG->wwwroot.'/userpix/upgrade.php'));
+
     require_login();
 
     require_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM));