svn: /pear/pearweb/trunk/public_html/pepr/ pepr-proposal-delete.php
| From: | Daniel O'Connor | Date: | Tue, 12 Apr 2011 23:37:37 +0000 |
| Subject: | svn: /pear/pearweb/trunk/public_html/pepr/ pepr-proposal-delete.php | ||
| Groups: | php.pear.cvs php.pear.core | ||
| Request: | Send a blank email to pear-core+get-9869@lists.php.net to get a copy of this message | ||
clockwerx Tue, 12 Apr 2011 23:37:37 +0000
Revision: http://svn.php.net/viewvc?view=revision&revision=310171
Log:
Swap to QuickForm2
Changed paths:
U pear/pearweb/trunk/public_html/pepr/pepr-proposal-delete.php
Modified: pear/pearweb/trunk/public_html/pepr/pepr-proposal-delete.php
===================================================================
--- pear/pearweb/trunk/public_html/pepr/pepr-proposal-delete.php 2011-04-12 22:28:23 UTC (rev
310170)
+++ pear/pearweb/trunk/public_html/pepr/pepr-proposal-delete.php 2011-04-12 23:37:37 UTC (rev
310171)
@@ -24,6 +24,7 @@
* Obtain the common functions and classes.
*/
require_once 'pepr/pepr.php';
+require_once 'HTML/QuickForm2.php';
auth_require('pear.pepr');
@@ -38,7 +39,7 @@
exit;
}
-if (!$proposal =& proposal::get($dbh, @$_GET['id'])) {
+if (empty($_GET['id'])) {
response_header('PEPr :: Delete :: Invalid Request');
echo "<h1>Delete Proposal</h1>\n";
report_error('The requested proposal does not exist.');
@@ -46,6 +47,20 @@
exit;
}
+$proposal = proposal::get($dbh, $_GET['id']);
+
+if (!$proposal) {
+ response_header('PEPr :: Delete :: Invalid Request');
+ echo "<h1>Delete Proposal</h1>\n";
+ report_error('The requested proposal does not exist.');
+ response_footer();
+ exit;
+}
+
+$karma = new Damblan_Karma($dbh);
+
+$form = new HTML_QuickForm2('delete-proposal', 'post', array('action'
=> 'pepr-proposal-delete.php?id=' . $proposal->id));
+
ob_start();
response_header('PEPr :: Delete :: ' . htmlspecialchars($proposal->pkg_name));
@@ -62,7 +77,6 @@
}
if ($proposal->compareStatus('>', 'proposal')) {
- $karma =& new Damblan_Karma($dbh);
if ($karma->has($auth_user->handle, 'pear.pepr.admin')) {
report_error('This proposal has reached the "'
. $proposal->getStatus(true) . '" phase.'
@@ -71,30 +85,22 @@
}
}
-include_once 'HTML/QuickForm.php';
-$form =& new HTML_QuickForm('delete-proposal', 'post',
- 'pepr-proposal-delete.php?id=' . $proposal->id);
$form->removeAttribute('name');
-$form->addElement('checkbox', 'delete', 'Really delete proposal for
',
- htmlspecialchars($proposal->pkg_category) . '::'
- . htmlspecialchars($proposal->pkg_name));
+$form->addElement('checkbox', 'delete', array('required' =>
'required'))
+ ->setLabel('You are sure?');
-$form->addElement('textarea', 'reason',
- 'Please tell us why you chose to delete this proposal ');
+$reason = $form->addElement('textarea', 'reason');
+$reason->setLabel('Why?');
$form->addElement('submit', 'submit', 'Do it');
-$form->addRule('delete', 'You have to check the box to delete!',
'required',
- '', 'client');
-
-
if (isset($_POST['submit'])) {
if ($form->validate()) {
$proposal->delete($dbh);
$proposal->sendActionEmail('proposal_delete', 'mixed',
$auth_user->handle,
- $form->exportValue('reason'));
+ $reason->getValue());
ob_end_clean();
localRedirect('pepr-proposal-delete.php?id=' . $proposal->id .
'&isDeleted=1');
} else {
@@ -106,7 +112,7 @@
ob_end_flush();
display_pepr_nav($proposal);
-$form->display();
+print $form;
response_footer();