mirror of
https://codeberg.org/streams/streams.git
synced 2024-09-20 03:15:19 +00:00
Merge pull request #558 from dawnbreak/dev
[BUGFIX] for fromStandalonePermission().
This commit is contained in:
commit
edde785219
2 changed files with 136 additions and 28 deletions
|
@ -12,22 +12,25 @@ require_once("include/text.php");
|
|||
* permission settings for an item with an empty ACL.
|
||||
* i.e the caption, icon, and tooltip for the no-ACL option in the ACL dialog.
|
||||
*/
|
||||
class PermissionDescription {
|
||||
class PermissionDescription {
|
||||
|
||||
private $global_perm;
|
||||
private $channel_perm;
|
||||
private $fallback_description;
|
||||
|
||||
|
||||
/**
|
||||
* Constructor is private.
|
||||
* Use static methods fromGlobalPermission(), fromStandalonePermission(), or fromDescription()
|
||||
* to create instances.
|
||||
* Use static methods fromGlobalPermission(), fromStandalonePermission(),
|
||||
* or fromDescription() to create instances.
|
||||
*
|
||||
* @internal
|
||||
* @param int $global_perm
|
||||
* @param int $channel_perm
|
||||
* @param string $description (optional) default empty
|
||||
*/
|
||||
private function __construct($global_perm, $channel_perm, $description = '') {
|
||||
|
||||
$this->global_perm = $global_perm;
|
||||
$this->channel_perm = $channel_perm;
|
||||
|
||||
$this->fallback_description = ($description == '') ? t('Visible to your default audience') : $description;
|
||||
}
|
||||
|
||||
|
@ -43,23 +46,22 @@ class PermissionDescription {
|
|||
return new PermissionDescription('', 0x80000, $description);
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Use this method only if the interpretation of an empty ACL doesn't fall back to a global
|
||||
* default permission. You should pass one of the constants from boot.php - PERMS_PUBLIC,
|
||||
* PERMS_NETWORK etc.
|
||||
*
|
||||
*
|
||||
* @param integer $perm - a single enumerated constant permission - PERMS_PUBLIC, PERMS_NETWORK etc.
|
||||
* @return a new instance of PermissionDescription
|
||||
*/
|
||||
public static function fromStandalonePermission($perm) {
|
||||
|
||||
$result = new PermissionDescription('', $perm);
|
||||
|
||||
$checkPerm = $this->get_permission_description();
|
||||
if ($checkPerm == $this->fallback_description) {
|
||||
|
||||
$checkPerm = $result->get_permission_description();
|
||||
if($checkPerm == $result->fallback_description) {
|
||||
$result = null;
|
||||
logger('null PermissionDescription from unknown standalone permission: ' . $perm ,LOGGER_DEBUG, LOG_ERROR);
|
||||
logger('null PermissionDescription from unknown standalone permission: ' . $perm, LOGGER_DEBUG, LOG_ERR);
|
||||
}
|
||||
|
||||
return $result;
|
||||
|
@ -67,9 +69,9 @@ class PermissionDescription {
|
|||
|
||||
/**
|
||||
* This is the preferred way to create a PermissionDescription, as it provides the most details.
|
||||
* Use this method if you know an empty ACL will result in one of the global default permissions
|
||||
* Use this method if you know an empty ACL will result in one of the global default permissions
|
||||
* being used, such as channel_r_stream (for which you would pass 'view_stream').
|
||||
*
|
||||
*
|
||||
* @param string $permname - a key for the global perms array from get_perms() in permissions.php,
|
||||
* e.g. 'view_stream', 'view_profile', etc.
|
||||
* @return a new instance of PermissionDescription
|
||||
|
@ -80,19 +82,19 @@ class PermissionDescription {
|
|||
|
||||
$global_perms = \Zotlabs\Access\Permissions::Perms();
|
||||
|
||||
if (array_key_exists($permname, $global_perms)) {
|
||||
if(array_key_exists($permname, $global_perms)) {
|
||||
|
||||
$channelPerm = \Zotlabs\Access\PermissionLimits::Get(\App::$channel['channel_id'],$permname);
|
||||
$channelPerm = \Zotlabs\Access\PermissionLimits::Get(\App::$channel['channel_id'], $permname);
|
||||
|
||||
$result = new PermissionDescription('', $channelPerm);
|
||||
} else {
|
||||
// The acl dialog can handle null arguments, but it shouldn't happen
|
||||
logger('null PermissionDescription from unknown global permission: ' . $permname ,LOGGER_DEBUG, LOG_ERROR);
|
||||
logger('null PermissionDescription from unknown global permission: ' . $permname, LOGGER_DEBUG, LOG_ERR);
|
||||
}
|
||||
|
||||
return $result;
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Gets a localized description of the permission, or a generic message if the permission
|
||||
* is unknown.
|
||||
|
@ -101,8 +103,7 @@ class PermissionDescription {
|
|||
*/
|
||||
public function get_permission_description() {
|
||||
|
||||
switch($this->channel_perm) {
|
||||
|
||||
switch($this->channel_perm) {
|
||||
case 0: return t('Only me');
|
||||
case PERMS_PUBLIC: return t('Public');
|
||||
case PERMS_NETWORK: return t('Anybody in the $Projectname network');
|
||||
|
@ -117,19 +118,18 @@ class PermissionDescription {
|
|||
|
||||
/**
|
||||
* Returns an icon css class name if an appropriate one is available, e.g. "fa-globe" for Public,
|
||||
* otherwise returns empty string.
|
||||
* otherwise returns empty string.
|
||||
*
|
||||
* @return string icon css class name (often FontAwesome)
|
||||
*/
|
||||
public function get_permission_icon() {
|
||||
|
||||
switch($this->channel_perm) {
|
||||
|
||||
switch($this->channel_perm) {
|
||||
case 0:/* only me */ return 'fa-eye-slash';
|
||||
case PERMS_PUBLIC: return 'fa-globe';
|
||||
case PERMS_NETWORK: return 'fa-share-alt-square'; // fa-share-alt-square is very similiar to the hubzilla logo, but we should create our own logo class to use
|
||||
case PERMS_SITE: return 'fa-sitemap';
|
||||
case PERMS_CONTACTS: return 'fa-group';
|
||||
case PERMS_SITE: return 'fa-sitemap';
|
||||
case PERMS_CONTACTS: return 'fa-group';
|
||||
case PERMS_SPECIFIC: return 'fa-list';
|
||||
case PERMS_AUTHED: return '';
|
||||
case PERMS_PENDING: return '';
|
||||
|
@ -137,7 +137,6 @@ class PermissionDescription {
|
|||
}
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* Returns a localized description of where the permission came from, if this is known.
|
||||
* If it's not know, or if the permission is standalone and didn't come from a default
|
||||
|
@ -147,8 +146,7 @@ class PermissionDescription {
|
|||
*/
|
||||
public function get_permission_origin_description() {
|
||||
|
||||
switch($this->global_perm) {
|
||||
|
||||
switch($this->global_perm) {
|
||||
case PERMS_R_STREAM: return t('This is your default setting for the audience of your normal stream, and posts.');
|
||||
case PERMS_R_PROFILE: return t('This is your default setting for who can view your default channel profile');
|
||||
case PERMS_R_ABOOK: return t('This is your default setting for who can view your connections');
|
||||
|
|
110
tests/unit/Lib/PermissionDescriptionTest.php
Normal file
110
tests/unit/Lib/PermissionDescriptionTest.php
Normal file
|
@ -0,0 +1,110 @@
|
|||
<?php
|
||||
/*
|
||||
* Copyright (c) 2016 Hubzilla
|
||||
*
|
||||
* Permission is hereby granted, free of charge, to any person obtaining a copy
|
||||
* of this software and associated documentation files (the "Software"), to deal
|
||||
* in the Software without restriction, including without limitation the rights
|
||||
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
||||
* copies of the Software, and to permit persons to whom the Software is
|
||||
* furnished to do so, subject to the following conditions:
|
||||
*
|
||||
* The above copyright notice and this permission notice shall be included in
|
||||
* all copies or substantial portions of the Software.
|
||||
*
|
||||
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
||||
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
||||
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
||||
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
||||
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
||||
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
||||
* SOFTWARE.
|
||||
*/
|
||||
|
||||
// Global namespace for fully qualified \App class.
|
||||
namespace {
|
||||
// General channel permissions in boot.php
|
||||
// 0 = Only you
|
||||
define ( 'PERMS_PUBLIC' , 0x0001 ); // anybody
|
||||
define ( 'PERMS_NETWORK' , 0x0002 ); // anybody in this network
|
||||
define ( 'PERMS_SITE' , 0x0004 ); // anybody on this site
|
||||
define ( 'PERMS_CONTACTS' , 0x0008 ); // any of my connections
|
||||
define ( 'PERMS_SPECIFIC' , 0x0080 ); // only specific connections
|
||||
define ( 'PERMS_AUTHED' , 0x0100 ); // anybody authenticated (could include visitors from other networks)
|
||||
define ( 'PERMS_PENDING' , 0x0200 ); // any connections including those who haven't yet been approved
|
||||
// log levels in boot.php
|
||||
define ( 'LOGGER_DEBUG', 2 );
|
||||
|
||||
// Stub global fully qualified \App class for static function calls
|
||||
class App {
|
||||
// Stub get_hostname()
|
||||
public static function get_hostname() {
|
||||
return 'phpunit';
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// Stub global functions used in PermissionDescription with the help of
|
||||
// PHP's namespace resolution rules.
|
||||
namespace Zotlabs\Lib {
|
||||
// Stub global translate function t()
|
||||
function t($s) {
|
||||
return $s;
|
||||
}
|
||||
// Stub global log function logger()
|
||||
function logger($msg, $level = LOGGER_NORMAL, $priority = LOG_INFO) {
|
||||
// doesn't matter
|
||||
}
|
||||
}
|
||||
|
||||
// regular namespace for this unit test
|
||||
namespace Zotlabs\Tests\Unit\Lib {
|
||||
|
||||
use Zotlabs\Tests\Unit\UnitTestCase;
|
||||
use Zotlabs\Lib\PermissionDescription;
|
||||
|
||||
/**
|
||||
* @brief Unit Test case for ConnectionPool class.
|
||||
*/
|
||||
class PermissionDescriptionTest extends UnitTestCase {
|
||||
|
||||
public function testFromDescription() {
|
||||
$permDesc = PermissionDescription::fromDescription('test');
|
||||
$permDesc2 = PermissionDescription::fromDescription('test');
|
||||
$permDesc3 = PermissionDescription::fromDescription('test2');
|
||||
|
||||
$this->assertEquals($permDesc, $permDesc2);
|
||||
$this->assertNotEquals($permDesc, $permDesc3);
|
||||
}
|
||||
|
||||
public function testFromStandalonePermission() {
|
||||
$permDescUnknown = PermissionDescription::fromStandalonePermission(-1);
|
||||
$permDescSelf = PermissionDescription::fromStandalonePermission(0);
|
||||
|
||||
$this->assertNull($permDescUnknown);
|
||||
$this->assertNotNull($permDescSelf);
|
||||
}
|
||||
|
||||
public function testFromGlobalPermission() {
|
||||
//$permDesc = PermissionDescription::fromGlobalPermission('view_profile');
|
||||
|
||||
$this->markTestIncomplete(
|
||||
'For this test we need more stubs...'
|
||||
);
|
||||
}
|
||||
|
||||
public function testGetPermissionDescription() {
|
||||
|
||||
// fromStandalonePermission uses get_permission_description(), so that will not help
|
||||
//$permDescSelf = PermissionDescription::fromStandalonePermission(0);
|
||||
//$permDescPublic = PermissionDescription::fromStandalonePermission(PERMS_PUBLIC);
|
||||
|
||||
$this->markTestIncomplete(
|
||||
'For this test we need a mock of PermissionDescription...'
|
||||
);
|
||||
//$permDescSelf =
|
||||
//$this->assertEquals($permDescSelf->, 'Only me');
|
||||
//$this->assertEquals($permDescPublic, 'Public');
|
||||
}
|
||||
}
|
||||
}
|
Loading…
Reference in a new issue