Issues in hdnewsadd.php - All Issues - Inspection of "Apply fixes from StyleCI" - aepnat/simhelpdesk - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#1)

by Saepul

created 2018-12-01 15:04 UTC

hdnewsadd.php (1 issue)

Labels

Bug 1

Severity

Major 1

<?php 
require 'core/init.php';
$general->logged_out_protect();

if (isset($_POST['submit'])) {
    $newsdate = strtotime($_POST['newsdate']);
    $title = $_POST['title'];
    $detail = $_POST['detail'];
    $user = $users->userdata($_SESSION['loginid']);
    $createdby = ucwords(strtolower($user['fullname']));
    $createdon = strtotime(now);

    $expired = strtotime($_POST['expireddate']);

    $hdnews->add_news($newsdate, $title, $detail, $createdby, $createdon, $expired);
    header('Location: hdnews.php');
}
?>
<!DOCTYPE HTML>
<html>
<head>
	<title>Add Helpdesk News</title>
	<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
	<style type="text/css">
		body{font-size:12px;background-image:url('images/corner.jpg');background-repeat:no-repeat;background-attachment:fixed;font-family: Arial, Helvetica, sans-serif;}
		.breadcrumb{font-size:12px;color:#0000A0;}
		.formtable {font-size:12px;color:#000000; background-color:#f0f0f0;padding:10px;width:600px;}
		.errormsg {font-size:10pt;color:#ff0000;text-align:left;}
	</style>
	<link rel="stylesheet" href="css/jquery-ui.css" />
	<script type="text/javascript" src="js/jquery.js"></script>
	<script src="js/jquery-ui.js"></script>
	<script type="text/javascript"> 
		$(document).ready(function(){
			$("#newsdate,#expireddate").datepicker
			({dateFormat:"dd-M-yy",changeMonth:true,changeYear:true,});
		});
	</script>
	<script type="text/javascript">
	function cekData()
	{	if (newsform.title.value == "")
		{	alert("Headline News must be filled!");
			newsform.title.focus();
			return false;
		}
		if (newsform.detail.value == "")
		{	alert("Detail News must be filled!");
			newsform.detail.focus();
			return false;
		}
		else
			return true;   
	}
	</script>
</head>
<body>	
	<div class="breadcrumb"> >> Admin >> Helpdesk News >> Add News</div>
	<hr/>
	<form name="newsform" method="post" action="" onsubmit="return cekData();">
	<fieldset style="display: inline-block;">
	<legend> Add Helpdesk News </legend>
	<table class="formtable">
		<tr align="left">
			<td> News Date </td><td> : </td>
			<td><input type="text" id="newsdate" name="newsdate" readonly="readonly" value="<?php echo date('d-M-Y', time()); ?>">
			</td>
		</tr>
		<tr align="left">
			<td> Headline News</td><td> : </td>
			<td> <input type='text' size='70' name='title' maxlength="70"> </td>
		</tr>
		<tr align="left">
			<td> News Detail </td><td> : </td>
			<td> <textarea name="detail" rows="3" cols="53"></textarea> </td>
		</tr>
		<tr align="left">
			<td> News Expired Date</td><td> : </td>
			<td><input type="text" id="expireddate" name="expireddate" readonly="readonly" value="<?php echo date('d-M-Y', time()); ?>">
			</td>
		</tr>
		<tr align="left">
			<td> </td>
			<td> </td>
			<td> <br/>
				<input type='submit' name='submit' value=' Submit '>  &nbsp;&nbsp;&nbsp;
				<input type='reset' name='reset' value=' Reset '> 
			</td>
		</tr>
	</table>
	</fieldset>
	</form>

	<?php 
    if (empty($errors) === false) {
        echo '<p class=errormsg>'.implode('</p><p class=errormsg>', $errors).'</p>';
    }
    ?>
</body>
</html>

1			<?php
2			require 'core/init.php';
3			$general->logged_out_protect();
4
5			if (isset($_POST['submit'])) {
6			$newsdate = strtotime($_POST['newsdate']);
7			$title = $_POST['title'];
8			$detail = $_POST['detail'];
9			$user = $users->userdata($_SESSION['loginid']);
10			$createdby = ucwords(strtolower($user['fullname']));
11			$createdon = strtotime(now);
			0 ignored issues – show Bug introduced 2018-12-01 14:58 UTC by Report Bug Copy Issue Report Show Similar Issues like this The constant `now` was not found. Maybe you did not declare it correctly or list all dependencies? Loading history...
12			$expired = strtotime($_POST['expireddate']);
13
14			$hdnews->add_news($newsdate, $title, $detail, $createdby, $createdon, $expired);
15			header('Location: hdnews.php');
16			}
17			?>
18			<!DOCTYPE HTML>
19			<html>
20			<head>
21			<title>Add Helpdesk News</title>
22			<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
23			<style type="text/css">
24			body{font-size:12px;background-image:url('images/corner.jpg');background-repeat:no-repeat;background-attachment:fixed;font-family: Arial, Helvetica, sans-serif;}
25			.breadcrumb{font-size:12px;color:#0000A0;}
26			.formtable {font-size:12px;color:#000000; background-color:#f0f0f0;padding:10px;width:600px;}
27			.errormsg {font-size:10pt;color:#ff0000;text-align:left;}
28			</style>
29			<link rel="stylesheet" href="css/jquery-ui.css" />
30			<script type="text/javascript" src="js/jquery.js"></script>
31			<script src="js/jquery-ui.js"></script>
32			<script type="text/javascript">
33			$(document).ready(function(){
34			$("#newsdate,#expireddate").datepicker
35			({dateFormat:"dd-M-yy",changeMonth:true,changeYear:true,});
36			});
37			</script>
38			<script type="text/javascript">
39			function cekData()
40			{ if (newsform.title.value == "")
41			{ alert("Headline News must be filled!");
42			newsform.title.focus();
43			return false;
44			}
45			if (newsform.detail.value == "")
46			{ alert("Detail News must be filled!");
47			newsform.detail.focus();
48			return false;
49			}
50			else
51			return true;
52			}
53			</script>
54			</head>
55			<body>
56			<div class="breadcrumb"> >> Admin >> Helpdesk News >> Add News</div>
57			<hr/>
58			<form name="newsform" method="post" action="" onsubmit="return cekData();">
59			<fieldset style="display: inline-block;">
60			<legend> Add Helpdesk News </legend>
61			<table class="formtable">
62			<tr align="left">
63			<td> News Date </td><td> : </td>
64			<td><input type="text" id="newsdate" name="newsdate" readonly="readonly" value="<?php echo date('d-M-Y', time()); ?>">
65			</td>
66			</tr>
67			<tr align="left">
68			<td> Headline News</td><td> : </td>
69			<td> <input type='text' size='70' name='title' maxlength="70"> </td>
70			</tr>
71			<tr align="left">
72			<td> News Detail </td><td> : </td>
73			<td> <textarea name="detail" rows="3" cols="53"></textarea> </td>
74			</tr>
75			<tr align="left">
76			<td> News Expired Date</td><td> : </td>
77			<td><input type="text" id="expireddate" name="expireddate" readonly="readonly" value="<?php echo date('d-M-Y', time()); ?>">
78			</td>
79			</tr>
80			<tr align="left">
81			<td> </td>
82			<td> </td>
83			<td> <br/>
84			<input type='submit' name='submit' value=' Submit '>
85			<input type='reset' name='reset' value=' Reset '>
86			</td>
87			</tr>
88			</table>
89			</fieldset>
90			</form>
91
92			<?php
93			if (empty($errors) === false) {
94			echo '<p class=errormsg>'.implode('</p><p class=errormsg>', $errors).'</p>';
95			}
96			?>
97			</body>
98			</html>

aepnat / simhelpdesk

Pull Request — master (#1)

hdnewsadd.php (1 issue)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like