Passed
Pull Request — master (#5574)
by David
09:40
created

pm_form()   B

Complexity

Conditions 10
Paths 44

Size

Total Lines 79
Code Lines 60

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 10
eloc 60
c 0
b 0
f 0
nc 44
nop 3
dl 0
loc 79
rs 7.006

How to fix   Long Method    Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
// This file is part of BOINC.
3
// http://boinc.berkeley.edu
4
// Copyright (C) 2008 University of California
5
//
6
// BOINC is free software; you can redistribute it and/or modify it
7
// under the terms of the GNU Lesser General Public License
8
// as published by the Free Software Foundation,
9
// either version 3 of the License, or (at your option) any later version.
10
//
11
// BOINC is distributed in the hope that it will be useful,
12
// but WITHOUT ANY WARRANTY; without even the implied warranty of
13
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
14
// See the GNU Lesser General Public License for more details.
15
//
16
// You should have received a copy of the GNU Lesser General Public License
17
// along with BOINC.  If not, see <http://www.gnu.org/licenses/>.
18
19
require_once("boinc_db.inc");
20
require_once("sanitize_html.inc");
21
require_once("bbcode_html.inc");
22
23
function pm_header() {
24
    echo "<div>\n";
25
    echo "    <a href=\"pm.php?action=inbox\">".tra("Inbox")."</a>\n";
26
    echo "    &middot; <a href=\"pm.php?action=new\">".tra("Write")."</a>\n";
27
    echo "</div>\n";
28
}
29
30
function pm_rules() {
31
    $x = "<table><tr><td align=left><small>";
32
    $x .= tra("
33
        <ul>
34
        <li> Messages may not contain content that is obscene, hate-related,
35
            sexually explicit or suggestive.
36
        <li> No commercial advertisements.
37
        <li> No links to web sites involving sexual content,
38
            gambling, or intolerance of others.
39
        <li> No messages intended to annoy or antagonize other people.
40
        <li> No messages that are deliberately hostile, threatening, or insulting.
41
        <li> No abusive comments involving race, religion,
42
            nationality, gender, class or sexuality.
43
        <li> The privileges of violators may be suspended or revoked.
44
        <li> If your account is suspended, don't create a new one.
45
        </ul>
46
    ");
47
    $x .= "</small></td></tr></table>\n";
48
    return $x;
49
}
50
51
function pm_team_form($user, $teamid, $error=null) {
52
    global $bbcode_html, $bbcode_js;
53
    $team = BoincTeam::lookup_id($teamid);
54
    if (!$team) {
55
        error_page("no such team");
56
    }
57
    if (!is_team_admin($user, $team)) {
58
        error_page("not admin");
59
    }
60
61
    page_head(tra("Send message to team"),'','','', $bbcode_js);
62
63
    $subject = post_str("subject", true);
64
    $content = post_str("content", true);
65
    if (post_str("preview", true) == tra("Preview")) {
66
        panel(tra('Preview'),
67
            function() use($content) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after USE keyword; found 0
Loading history...
68
                echo output_transform($content, null);
69
            }
70
        );
71
    }
72
    if ($error) {
73
        echo "<p class=\"text-danger\">".$error."</p>\n";
74
    }
75
76
    echo "<form action=\"pm.php\" method=\"post\" name=\"post\" onsubmit=\"return checkForm(this)\">\n";
77
    echo "<input type=\"hidden\" name=\"action\" value=\"send\">\n";
78
    echo "<input type=\"hidden\" name=\"teamid\" value=\"$teamid\">\n";
79
    echo form_tokens($user->authenticator);
80
    start_table();
81
    row2(
82
        tra("Subject"),
83
        "<input type=\"text\" class=\"form-control\" name=\"subject\" value=\"$subject\">",
84
        null, '20%'
85
    );
86
    row2_init(tra("Message")."<small>".bbcode_info()."</small>", '20%');
87
    start_table();
88
    echo $bbcode_html;
89
    echo "<tr><td>\n";
90
    echo "<textarea name=\"content\" class=\"form-control\" rows=\"18\">$content</textarea>";
91
    echo "</td></tr>\n";
92
    end_table();
93
    echo "<tr><td></td><td>
94
        <input class=\"btn btn-primary\" type=\"submit\" name=\"preview\" value=\"".tra("Preview")."\">
95
        <input class=\"btn btn-success\" type=\"submit\" value=\"".tra("Send message")."\">
96
        </td></tr>\n
97
    ";
98
    end_table();
99
    page_tail();
100
}
101
102
function pm_form($replyto, $userid, $error = null) {
103
    global $bbcode_html, $bbcode_js;
104
    global $g_logged_in_user;
105
    page_head(tra("Send private message"),'','','', $bbcode_js);
106
107
    if (post_str("preview", true) == tra("Preview")) {
108
        $content = post_str("content", true);
109
        panel(tra('Preview'),
110
            function() use($content) {
0 ignored issues
show
Coding Style introduced by
Expected 1 space after USE keyword; found 0
Loading history...
111
                echo output_transform($content, null);
112
            }
113
        );
114
    }
115
116
    $subject = null;
117
    $content = null;
118
    if ($replyto) {
119
        $message = BoincPrivateMessage::lookup_id($replyto);
120
        if (!$message || $message->userid != $g_logged_in_user->id) {
121
            error_page(tra("no such message"));
122
        }
123
        $content = "[quote]".$message->content."[/quote]\n";
124
        $userid = $message->senderid;
125
        $user = BoincUser::lookup_id($userid);
126
        if (!$user) {
127
            error_page("Sender no longer exists");
128
        }
129
        $writeto = $userid." (".$user->name.")";
130
        $subject = $message->subject;
131
        if (substr($subject, 0, 3) != "re:") {
132
            $subject = "re: ".$subject;
133
        }
134
    } elseif ($userid) {
135
        $user = BoincUser::lookup_id($userid);
136
        if (!$user) {
137
            error_page("Sender no longer exists");
138
        }
139
        $writeto = $userid." (".$user->name.")";
140
    } else {
141
        $writeto = sanitize_tags(post_str("to", true));
142
        $subject = post_str("subject", true);
143
        $content = post_str("content", true);
144
    }
145
146
    $content = htmlspecialchars($content);
147
    $subject = htmlspecialchars($subject);
148
149
    if ($error != null) {
150
        echo "<p class=\"text-danger\">".$error."</p>\n";
151
    }
152
153
    echo "<form action=\"pm.php\" method=\"post\" name=\"post\" onsubmit=\"return checkForm(this)\">\n";
154
    echo "<input type=\"hidden\" name=\"action\" value=\"send\">\n";
155
    echo form_tokens($g_logged_in_user->authenticator);
156
    start_table();
157
    row2(tra("To")."<br /><small>".tra("User IDs or unique usernames, separated with commas")."</small>",
158
        "<input type=\"text\" class=\"form-control\" name=\"to\" value=\"$writeto\">",
159
        null, '20%'
160
    );
161
    row2(
162
        tra("Subject"),
163
            "<input type=\"text\" class=\"form-control\" name=\"subject\" value=\"$subject\">",
164
        null, '20%'
165
    );
166
    row2_init(
167
        tra("Message")."<small>".bbcode_info().pm_rules()."</small>",
168
        '20%'
169
    );
170
    start_table();
171
    echo $bbcode_html;
172
    echo "<tr><td>\n";
173
    echo "<textarea name=\"content\" class=\"form-control\" rows=\"18\">$content</textarea>";
174
    echo "</td></tr>\n";
175
    end_table();
176
    echo "<tr><td></td><td><input class=\"btn btn-primary\" type=\"submit\" name=\"preview\" value=\"".tra("Preview")."\"> <input class=\"btn btn-success\" type=\"submit\" value=\"".tra("Send message")."\"></td></tr>\n";
177
    end_table();
178
179
    page_tail();
180
    exit();
0 ignored issues
show
Best Practice introduced by
Using exit here is not recommended.

In general, usage of exit should be done with care and only when running in a scripting context like a CLI script.

Loading history...
181
}
182
183
function send_pm_notification_email(
184
    $logged_in_user, $to_user, $subject, $content
0 ignored issues
show
Coding Style introduced by
Multi-line function declarations must define one parameter per line
Loading history...
185
) {
186
    $message  = "
187
You have received a new private message at ".PROJECT.".
188
189
From: $logged_in_user->name (ID $logged_in_user->id)
190
Subject: $subject
191
192
$content
193
194
--------------------------
195
To delete or respond to this message, visit:
196
".secure_url_base()."pm.php
197
198
To change email preferences, visit:
199
".secure_url_base()."edit_forum_preferences_form.php
200
Do not reply to this message.
201
" ;
0 ignored issues
show
Coding Style introduced by
Space found before semicolon; expected "";" but found "" ;"
Loading history...
202
    send_email($to_user, "[".PROJECT."] - private message", $message);
203
}
204
205
function pm_email_line($notify) {
206
    $pm = BoincPrivateMessage::lookup_id($notify->opaque);
207
    $from_user = BoincUser::lookup_id($pm->senderid);
208
    if (!$pm || !$from_user) return null;
209
    return "$from_user->name ".tra("sent you a private message; subject:")." '$pm->subject'";
210
}
211
212
function pm_web_line($notify) {
213
    $pm = BoincPrivateMessage::lookup_id($notify->opaque);
214
    $from_user = BoincUser::lookup_id($pm->senderid);
215
    if (!$pm || !$from_user) return null;
216
    return "<a href=pm.php>".tra("Private message%1 from %2, subject:" , "</a>", $from_user->name )." $pm->subject";
217
}
218
219
function pm_send_msg($from_user, $to_user, $subject, $content, $send_email) {
220
    $sql_subject = BoincDb::escape_string(sanitize_tags($subject));
221
    $sql_content = BoincDb::escape_string($content);
222
    $mid = BoincPrivateMessage::insert("(userid, senderid, date, subject, content) VALUES ($to_user->id, $from_user->id, UNIX_TIMESTAMP(), '$sql_subject', '$sql_content')");
223
    if (!$mid) {
224
        error_page(tra("Couldn't create message"));
225
    }
226
    // send email notification if needed
227
    //
228
    if ($send_email) {
229
        BoincForumPrefs::lookup($to_user);
230
        switch ($to_user->prefs->pm_notification) {
231
        case 0:
0 ignored issues
show
Coding Style introduced by
Empty CASE statements are not allowed
Loading history...
232
        case 2:
0 ignored issues
show
Coding Style introduced by
Empty CASE statements are not allowed
Loading history...
233
            break;
234
        case 1:
235
            send_pm_notification_email(
236
                $from_user, $to_user, $subject, $content
237
            );
238
            break;
239
        }
240
    }
241
242
    // create notification in any case
243
    //
244
    BoincNotify::insert("(userid, create_time, type, opaque) values ($to_user->id, ".time().", ".NOTIFY_PM.", $mid)");
245
}
246
247
function pm_count($userid, $duration) {
248
    $time = time() - $duration;
249
250
    // we don't want to include team messages in this count.
251
    // Kludge for excluding them based on subject.
252
    // Should add a flag to private_message to distinguish them.
253
    //
254
    return BoincPrivateMessage::count(
255
        "senderid=$userid AND date>$time AND subject not like 'Message from team%'"
256
    );
257
}
258
259
function check_pm_count($userid) {
260
    if ((pm_count($userid, 60) >= 2) || (pm_count($userid, 600) >= 5) ||
261
        (pm_count($userid, 3600) >= 15) || (pm_count($userid, 86400) >= 50)) {
262
        error_page(tra("You are not allowed to send privates messages so often. Please wait some time before sending more messages."));
263
    }
264
}
265
266
function pm_notification($user) {
267
    $output = "";
268
    $unread = BoincPrivateMessage::count("userid=$user->id AND opened=0");
269
270
    $output .= "<a href=\"pm.php?action=inbox\">".tra("Inbox")."</a>";
271
    if ($unread) {
272
        $output .= "<span class=\"inboxunread\"> ($unread ".tra("unread").")</span>\n";
273
    }
274
    $output .= " &middot; <a href=\"pm.php?action=new\">".tra("Write")."</a>\n";
275
    return $output;
276
}
277
278
function pm_email_remind($user) {
279
    if (!$user->prefs->pm_notification) {
280
        return "<br><small>" .
281
            tra(
282
                "For email notification, %1 edit community prefs %2",
283
                '<a href="edit_forum_preferences_form.php">', '</a>'
284
            ) .
285
            "</small>"
286
        ;
0 ignored issues
show
Coding Style introduced by
Space found before semicolon; expected ""</small>";" but found ""</small>"
;"
Loading history...
287
    }
288
    return "";
289
}
290
291
function pm_rss($notify, &$title, &$msg, &$url) {
292
    $pm = BoincPrivateMessage::lookup_id($notify->opaque);
293
    $from_user = BoincUser::lookup_id($pm->senderid);
294
    if (!$pm || !$from_user) {
295
        $msg = null;
296
        return;
297
    }
298
    $title = tra("Private message");
299
    $msg = "You have received a <a href=".secure_url_base()."pm.php>private message</a>.";
300
    $url = secure_url_base()."pm.php";
301
}
302
303
function pm_delete_user($user) {
304
    $mm = BoincPrivateMessage::enum("userid=$user->id or senderid=$user->id");
305
    foreach ($mm as $m) {
306
        $m->delete();
307
    }
308
}
309
310
$cvs_version_tracker[]="\$Id: pm.inc 14019 2007-11-01 23:04:39Z davea $";
311
?>
312