Skip to content

Commit 66bb9ea

Browse files
Frederic Massartdanpoltawski
authored andcommitted
MDL-55497 mod_forum: Converted quick forum search to a renderable
Part of MDL-55071
1 parent d7c6575 commit 66bb9ea

File tree

5 files changed

+116
-16
lines changed

5 files changed

+116
-16
lines changed
Lines changed: 75 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,75 @@
1+
<?php
2+
// This file is part of Moodle - http://moodle.org/
3+
//
4+
// Moodle is free software: you can redistribute it and/or modify
5+
// it under the terms of the GNU General Public License as published by
6+
// the Free Software Foundation, either version 3 of the License, or
7+
// (at your option) any later version.
8+
//
9+
// Moodle is distributed in the hope that it will be useful,
10+
// but WITHOUT ANY WARRANTY; without even the implied warranty of
11+
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12+
// GNU General Public License for more details.
13+
//
14+
// You should have received a copy of the GNU General Public License
15+
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.
16+
17+
/**
18+
* Quick search form renderable.
19+
*
20+
* @package mod_forum
21+
* @copyright 2016 Frédéric Massart - FMCorz.net
22+
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
23+
*/
24+
25+
namespace mod_forum\output;
26+
defined('MOODLE_INTERNAL') || die();
27+
28+
use help_icon;
29+
use moodle_url;
30+
use renderable;
31+
use renderer_base;
32+
use templatable;
33+
34+
/**
35+
* Quick search form renderable class.
36+
*
37+
* @package mod_forum
38+
* @copyright 2016 Frédéric Massart - FMCorz.net
39+
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
40+
*/
41+
class quick_search_form implements renderable, templatable {
42+
43+
/** @var int The course ID. */
44+
protected $courseid;
45+
/** @var string Current query. */
46+
protected $query;
47+
/** @var moodle_url The form action URL. */
48+
protected $actionurl;
49+
/** @var help_icon The help icon. */
50+
protected $helpicon;
51+
52+
/**
53+
* Constructor.
54+
*
55+
* @param int $courseid The course ID.
56+
* @param string $query The current query.
57+
*/
58+
public function __construct($courseid, $query = '') {
59+
$this->courseid = $courseid;
60+
$this->query = $query;
61+
$this->actionurl = new moodle_url('/mod/forum/search.php');
62+
$this->helpicon = new help_icon('search', 'core');
63+
}
64+
65+
public function export_for_template(renderer_base $output) {
66+
$data = [
67+
'actionurl' => $this->actionurl->out(false),
68+
'courseid' => $this->courseid,
69+
'query' => $this->query,
70+
'helpicon' => $this->helpicon->export_for_template($output),
71+
];
72+
return $data;
73+
}
74+
75+
}

mod/forum/lib.php

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3928,22 +3928,10 @@ function forum_print_mode_form($id, $mode, $forumtype='') {
39283928
* @return string
39293929
*/
39303930
function forum_search_form($course, $search='') {
3931-
global $CFG, $OUTPUT;
3932-
3933-
$output = '<div class="forumsearch">';
3934-
$output .= '<form action="'.$CFG->wwwroot.'/mod/forum/search.php" style="display:inline">';
3935-
$output .= '<fieldset class="invisiblefieldset">';
3936-
$output .= $OUTPUT->help_icon('search');
3937-
$output .= '<label class="accesshide" for="search" >'.get_string('search', 'forum').'</label>';
3938-
$output .= '<input id="search" name="search" type="text" size="18" value="'.s($search, true).'" />';
3939-
$output .= '<label class="accesshide" for="searchforums" >'.get_string('searchforums', 'forum').'</label>';
3940-
$output .= '<input id="searchforums" value="'.get_string('searchforums', 'forum').'" type="submit" />';
3941-
$output .= '<input name="id" type="hidden" value="'.$course->id.'" />';
3942-
$output .= '</fieldset>';
3943-
$output .= '</form>';
3944-
$output .= '</div>';
3945-
3946-
return $output;
3931+
global $CFG, $PAGE;
3932+
$forumsearch = new \mod_forum\output\quick_search_form($course->id, $search);
3933+
$output = $PAGE->get_renderer('mod_forum');
3934+
return $output->render($forumsearch);
39473935
}
39483936

39493937

mod/forum/renderer.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,4 +226,14 @@ public function render_digest_options($forum, $value) {
226226

227227
return $editable;
228228
}
229+
230+
/**
231+
* Render quick search form.
232+
*
233+
* @param \mod_forum\output\quick_search_form $form The renderable.
234+
* @return string
235+
*/
236+
public function render_quick_search_form(\mod_forum\output\quick_search_form $form) {
237+
return $this->render_from_template('mod_forum/quick_search_form', $form->export_for_template($this));
238+
}
229239
}
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
<div class="forumsearch">
2+
<form action="{{actionurl}}" style="display: inline;">
3+
<fieldset class="invisiblefieldset">
4+
{{#helpicon}}
5+
{{>core/help_icon}}
6+
{{/helpicon}}
7+
<input type="hidden" name="id" value="{{courseid}}">
8+
<label class="accesshide" for="search">{{#str}}search, forum{{/str}}</label>
9+
<input id="search" name="search" type="text" size="18" value="{{query}}">
10+
<label class="accesshide" for="searchforums">{{#str}}searchforums, forum{{/str}}</label>
11+
<input id="searchforums" type="submit" value={{#quote}}{{#str}}searchforums, forum{{/str}}{{/quote}}>
12+
</fieldset>
13+
</form>
14+
</div>
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
<div class="forumsearch">
2+
<form action="{{actionurl}}" class="form-inline">
3+
<input type="hidden" name="id" value="{{courseid}}">
4+
<div class="form-group">
5+
{{#helpicon}}
6+
{{>core/help_icon}}
7+
{{/helpicon}}
8+
<label class="sr-only" for="search">{{#str}}search, forum{{/str}}</label>
9+
<input id="search" name="search" type="text" class="form-control" value="{{query}}">
10+
</div>
11+
<button class="btn btn-secondary" id="searchforums" type="submit">{{#str}}searchforums, mod_forum{{/str}}</button>
12+
</form>
13+
</div>

0 commit comments

Comments
 (0)