Commit dcb343c3efc2dfbd476acfa10aceb81869320021
1 parent
4079982f
added validation and edited layout
git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@1663 c91229c3-7414-0410-bfa2-8a42b809f60b
Showing
3 changed files
with
11 additions
and
17 deletions
presentation/lookAndFeel/knowledgeTree/administration/rolemanagement/addRoleBL.php
| @@ -26,6 +26,7 @@ if (checkSession()) { | @@ -26,6 +26,7 @@ if (checkSession()) { | ||
| 26 | $oPatternCustom->setHtml(getPage()); | 26 | $oPatternCustom->setHtml(getPage()); |
| 27 | $main->setCentralPayload($oPatternCustom); | 27 | $main->setCentralPayload($oPatternCustom); |
| 28 | $main->setFormAction("$default->rootUrl/presentation/lookAndFeel/knowledgeTree/create.php?fRedirectURL=".urlencode("$default->rootUrl/control.php?action=editRole&fFromCreate=1&fRoleID=")); | 28 | $main->setFormAction("$default->rootUrl/presentation/lookAndFeel/knowledgeTree/create.php?fRedirectURL=".urlencode("$default->rootUrl/control.php?action=editRole&fFromCreate=1&fRoleID=")); |
| 29 | + $main->setHasRequiredFields(true); | ||
| 29 | $main->render(); | 30 | $main->render(); |
| 30 | } | 31 | } |
| 31 | ?> | 32 | ?> |
presentation/lookAndFeel/knowledgeTree/administration/rolemanagement/editRoleBL.php
| @@ -76,6 +76,7 @@ if (checkSession()) { | @@ -76,6 +76,7 @@ if (checkSession()) { | ||
| 76 | } | 76 | } |
| 77 | //render the page | 77 | //render the page |
| 78 | $main->setCentralPayload($oPatternCustom); | 78 | $main->setCentralPayload($oPatternCustom); |
| 79 | + $main->setHasRequiredFields(true); | ||
| 79 | $main->render(); | 80 | $main->render(); |
| 80 | } | 81 | } |
| 81 | ?> | 82 | ?> |
presentation/lookAndFeel/knowledgeTree/administration/rolemanagement/editRoleUI.inc
| @@ -12,7 +12,6 @@ function getCreatePage($iRoleID) { | @@ -12,7 +12,6 @@ function getCreatePage($iRoleID) { | ||
| 12 | 12 | ||
| 13 | global $default; | 13 | global $default; |
| 14 | 14 | ||
| 15 | - | ||
| 16 | $sQuery = "SELECT * FROM $default->owl_roles_table WHERE id = $iRoleID"; | 15 | $sQuery = "SELECT * FROM $default->owl_roles_table WHERE id = $iRoleID"; |
| 17 | $aDisplayColumns = array("name","can_read","can_write"); | 16 | $aDisplayColumns = array("name","can_read","can_write"); |
| 18 | $aStoreColumns = array("name","can_read","can_write"); | 17 | $aStoreColumns = array("name","can_read","can_write"); |
| @@ -27,7 +26,6 @@ function getCreatePage($iRoleID) { | @@ -27,7 +26,6 @@ function getCreatePage($iRoleID) { | ||
| 27 | 26 | ||
| 28 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; | 27 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; |
| 29 | $sToRender .= "<table>\n"; | 28 | $sToRender .= "<table>\n"; |
| 30 | - $sToRender .= "<br></br>\n"; | ||
| 31 | $sToRender .= "<tr></tr>\n"; | 29 | $sToRender .= "<tr></tr>\n"; |
| 32 | $sToRender .= "<tr>\n"; | 30 | $sToRender .= "<tr>\n"; |
| 33 | $sToRender .= "<td>" . $oPattern->render() . "</td>\n"; | 31 | $sToRender .= "<td>" . $oPattern->render() . "</td>\n"; |
| @@ -39,15 +37,10 @@ function getCreatePage($iRoleID) { | @@ -39,15 +37,10 @@ function getCreatePage($iRoleID) { | ||
| 39 | 37 | ||
| 40 | $sToRender .= "<td align = right><input type=\"image\" src =\"$default->graphicsUrl/widgets/update.gif\" value=\"Update\" border=\"0\"/><a href=\"$default->rootUrl/control.php?action=addRole\"><img src =\"$default->graphicsUrl/widgets/cancel.gif\" border = \"0\" /></a></td>\n"; | 38 | $sToRender .= "<td align = right><input type=\"image\" src =\"$default->graphicsUrl/widgets/update.gif\" value=\"Update\" border=\"0\"/><a href=\"$default->rootUrl/control.php?action=addRole\"><img src =\"$default->graphicsUrl/widgets/cancel.gif\" border = \"0\" /></a></td>\n"; |
| 41 | $sToRender .= "</table>\n"; | 39 | $sToRender .= "</table>\n"; |
| 42 | - | ||
| 43 | - | ||
| 44 | - | 40 | + |
| 45 | return $sToRender; | 41 | return $sToRender; |
| 46 | - | ||
| 47 | - | ||
| 48 | } | 42 | } |
| 49 | 43 | ||
| 50 | - | ||
| 51 | // if fails .. | 44 | // if fails .. |
| 52 | function getAddFailPage() { | 45 | function getAddFailPage() { |
| 53 | 46 | ||
| @@ -55,8 +48,6 @@ function getAddFailPage() { | @@ -55,8 +48,6 @@ function getAddFailPage() { | ||
| 55 | 48 | ||
| 56 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; | 49 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; |
| 57 | $sToRender .= "<table>\n"; | 50 | $sToRender .= "<table>\n"; |
| 58 | - $sToRender .= "<br></br>\n"; | ||
| 59 | - $sToRender .= "<tr><b>Add Role Failed:</b></tr>\n"; | ||
| 60 | $sToRender .= "<tr>\n"; | 51 | $sToRender .= "<tr>\n"; |
| 61 | $sToRender .= "<td>Sorry, Role Already Exists!</td>\n"; | 52 | $sToRender .= "<td>Sorry, Role Already Exists!</td>\n"; |
| 62 | $sToRender .= "</tr>\n"; | 53 | $sToRender .= "</tr>\n"; |
| @@ -68,7 +59,6 @@ function getAddFailPage() { | @@ -68,7 +59,6 @@ function getAddFailPage() { | ||
| 68 | $sToRender .= "</table>\n"; | 59 | $sToRender .= "</table>\n"; |
| 69 | 60 | ||
| 70 | return $sToRender; | 61 | return $sToRender; |
| 71 | - | ||
| 72 | } | 62 | } |
| 73 | 63 | ||
| 74 | // if its the manual edit page..then display normally | 64 | // if its the manual edit page..then display normally |
| @@ -77,14 +67,10 @@ function getEditPage($iRoleID) { | @@ -77,14 +67,10 @@ function getEditPage($iRoleID) { | ||
| 77 | $oRole = null; | 67 | $oRole = null; |
| 78 | if (isset($iRoleID)) { | 68 | if (isset($iRoleID)) { |
| 79 | $oRole = Role::get($iRoleID); | 69 | $oRole = Role::get($iRoleID); |
| 80 | - | ||
| 81 | } | 70 | } |
| 82 | 71 | ||
| 83 | - | ||
| 84 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; | 72 | $sToRender .= "<table width=\"600\">" . renderHeading("Edit Role") . "</table>"; |
| 85 | - $sToRender .= "<b>Edit Role Properties:</b>\n"; | ||
| 86 | $sToRender .= "<table>\n"; | 73 | $sToRender .= "<table>\n"; |
| 87 | - $sToRender .= "<br></br>\n"; | ||
| 88 | $sToRender .= "<tr></tr>\n"; | 74 | $sToRender .= "<tr></tr>\n"; |
| 89 | $sToRender .= "<tr>\n"; | 75 | $sToRender .= "<tr>\n"; |
| 90 | $sToRender .= "<td>Role Name: </td><td>" . getRoleDisplay($oRole) . "</td>\n"; | 76 | $sToRender .= "<td>Role Name: </td><td>" . getRoleDisplay($oRole) . "</td>\n"; |
| @@ -106,14 +92,20 @@ function getEditPage($iRoleID) { | @@ -106,14 +92,20 @@ function getEditPage($iRoleID) { | ||
| 106 | $sToRender .= "</tr>\n"; | 92 | $sToRender .= "</tr>\n"; |
| 107 | } | 93 | } |
| 108 | 94 | ||
| 109 | - | ||
| 110 | $sToRender .= "<tr>\n"; | 95 | $sToRender .= "<tr>\n"; |
| 111 | $sToRender .= "</tr>\n"; | 96 | $sToRender .= "</tr>\n"; |
| 112 | $sToRender .= "<td></td>" . getUpdateButton($oRole); | 97 | $sToRender .= "<td></td>" . getUpdateButton($oRole); |
| 113 | $sToRender .= getCancelButton($oRole); | 98 | $sToRender .= getCancelButton($oRole); |
| 114 | $sToRender .= "</table>\n"; | 99 | $sToRender .= "</table>\n"; |
| 115 | 100 | ||
| 116 | - | 101 | + $sToRender .= "\n\n<script language=\"javascript\">\n<!--\n"; |
| 102 | + $sToRender .= "function validateForm(theForm) {\n"; | ||
| 103 | + $sToRender .= "\tif (!(validRequired(theForm.fRoleName, 'Role Name'))) {\n"; | ||
| 104 | + $sToRender .= "\t\treturn false;\n"; | ||
| 105 | + $sToRender .= "\t}\n"; | ||
| 106 | + $sToRender .= "return true;\n}\n"; | ||
| 107 | + $sToRender .= "//-->\n</script>\n\n"; | ||
| 108 | + | ||
| 117 | return $sToRender; | 109 | return $sToRender; |
| 118 | } | 110 | } |
| 119 | 111 |