Check that users can view applications and versions before displaying them. Present an error message if they lack permission. Fixes a bug where we were allowing users to view applications and versions that were rejected.
This commit is contained in:
15
appview.php
15
appview.php
@@ -191,6 +191,13 @@ if($_REQUEST['appId'])
|
|||||||
{
|
{
|
||||||
$oApp = new Application($_REQUEST['appId']);
|
$oApp = new Application($_REQUEST['appId']);
|
||||||
|
|
||||||
|
/* is this user supposed to view this version? */
|
||||||
|
if(!$_SESSION['current']->canViewApplication($oApp))
|
||||||
|
{
|
||||||
|
errorpage("Something went wrong with the application or version id");
|
||||||
|
exit;
|
||||||
|
}
|
||||||
|
|
||||||
// show Vote Menu
|
// show Vote Menu
|
||||||
if($_SESSION['current']->isLoggedIn())
|
if($_SESSION['current']->isLoggedIn())
|
||||||
apidb_sidebar_add("vote_menu");
|
apidb_sidebar_add("vote_menu");
|
||||||
@@ -329,6 +336,14 @@ if($_REQUEST['appId'])
|
|||||||
else if($_REQUEST['versionId'])
|
else if($_REQUEST['versionId'])
|
||||||
{
|
{
|
||||||
$oVersion = new Version($_REQUEST['versionId']);
|
$oVersion = new Version($_REQUEST['versionId']);
|
||||||
|
|
||||||
|
/* is this user supposed to view this version? */
|
||||||
|
if(!$_SESSION['current']->canViewVersion($oVersion))
|
||||||
|
{
|
||||||
|
errorpage("Something went wrong with the application or version id");
|
||||||
|
exit;
|
||||||
|
}
|
||||||
|
|
||||||
$oApp = new Application($oVersion->iAppId);
|
$oApp = new Application($oVersion->iAppId);
|
||||||
if(!$oApp->iAppId)
|
if(!$oApp->iAppId)
|
||||||
{
|
{
|
||||||
|
|||||||
Reference in New Issue
Block a user