trim expects error ... in admin/site

This commit is contained in:
zotlabs 2018-03-10 13:20:20 -08:00
parent 3ee35b83c2
commit a417389934
3 changed files with 20 additions and 13 deletions

View File

@ -14,6 +14,8 @@ class Site {
return; return;
} }
logger('post: ' . print_r($_POST,true));
check_form_security_token_redirectOnErr('/admin/site', 'admin_site'); check_form_security_token_redirectOnErr('/admin/site', 'admin_site');
$sitename = ((x($_POST,'sitename')) ? notags(trim($_POST['sitename'])) : ''); $sitename = ((x($_POST,'sitename')) ? notags(trim($_POST['sitename'])) : '');
@ -38,7 +40,11 @@ class Site {
$site_sellpage = ((x($_POST,'site_sellpage')) ? notags(trim($_POST['site_sellpage'])) : ''); $site_sellpage = ((x($_POST,'site_sellpage')) ? notags(trim($_POST['site_sellpage'])) : '');
$site_location = ((x($_POST,'site_location')) ? notags(trim($_POST['site_location'])) : ''); $site_location = ((x($_POST,'site_location')) ? notags(trim($_POST['site_location'])) : '');
$frontpage = ((x($_POST,'frontpage')) ? notags(trim($_POST['frontpage'])) : ''); $frontpage = ((x($_POST,'frontpage')) ? notags(trim($_POST['frontpage'])) : '');
$firstpage = ((x(trim($_POST,'firstpage'))) ? notags(trim($_POST['firstpage'])) : 'profiles'); $first_page = ((x($_POST,'first_page')) ? notags(trim($_POST['first_page'])) : 'profiles');
// check value after trim
if(! $first_page) {
$first_page = 'profiles';
}
$mirror_frontpage = ((x($_POST,'mirror_frontpage')) ? intval(trim($_POST['mirror_frontpage'])) : 0); $mirror_frontpage = ((x($_POST,'mirror_frontpage')) ? intval(trim($_POST['mirror_frontpage'])) : 0);
$directory_server = ((x($_POST,'directory_server')) ? trim($_POST['directory_server']) : ''); $directory_server = ((x($_POST,'directory_server')) ? trim($_POST['directory_server']) : '');
$allowed_sites = ((x($_POST,'allowed_sites')) ? notags(trim($_POST['allowed_sites'])) : ''); $allowed_sites = ((x($_POST,'allowed_sites')) ? notags(trim($_POST['allowed_sites'])) : '');
@ -82,7 +88,7 @@ class Site {
set_config('system', 'maxloadavg', $maxloadavg); set_config('system', 'maxloadavg', $maxloadavg);
set_config('system', 'frontpage', $frontpage); set_config('system', 'frontpage', $frontpage);
set_config('system', 'sellpage', $site_sellpage); set_config('system', 'sellpage', $site_sellpage);
set_config('system', 'workflow_channel_next', $firstpage); set_config('system', 'workflow_channel_next', $first_page);
set_config('system', 'site_location', $site_location); set_config('system', 'site_location', $site_location);
set_config('system', 'mirror_frontpage', $mirror_frontpage); set_config('system', 'mirror_frontpage', $mirror_frontpage);
set_config('system', 'sitename', $sitename); set_config('system', 'sitename', $sitename);
@ -345,7 +351,7 @@ class Site {
'$default_expire_days' => array('default_expire_days', t('Expiration period in days for imported (grid/network) content'), intval(get_config('system','default_expire_days')), t('0 for no expiration of imported content')), '$default_expire_days' => array('default_expire_days', t('Expiration period in days for imported (grid/network) content'), intval(get_config('system','default_expire_days')), t('0 for no expiration of imported content')),
'$sellpage' => array('site_sellpage', t('Public servers: Optional landing (marketing) webpage for new registrants'), get_config('system','sellpage',''), sprintf( t('Create this page first. Default is %s/register'),z_root())), '$sellpage' => array('site_sellpage', t('Public servers: Optional landing (marketing) webpage for new registrants'), get_config('system','sellpage',''), sprintf( t('Create this page first. Default is %s/register'),z_root())),
'$firstpage' => array('firstpage', t('Page to display after creating a new channel'), get_config('system','workflow_channel_next','profiles'), t('Recommend: profiles, go, or settings')), '$first_page' => array('first_page', t('Page to display after creating a new channel'), get_config('system','workflow_channel_next','profiles'), t('Recommend: profiles, go, or settings')),
'$location' => array('site_location', t('Optional: site location'), get_config('system','site_location',''), t('Region or country')), '$location' => array('site_location', t('Optional: site location'), get_config('system','site_location',''), t('Region or country')),

View File

@ -114,15 +114,16 @@ class Import extends \Zotlabs\Web\Controller {
return; return;
} }
if(array_key_exists('compatibility',$data) && array_key_exists('database',$data['compatibility'])) { // This is only an info message but it is alarming to folks who then report failure with this as the cause, when in fact we ignore this completely.
$v1 = substr($data['compatibility']['database'],-4); // if(array_key_exists('compatibility',$data) && array_key_exists('database',$data['compatibility'])) {
$v2 = substr(DB_UPDATE_VERSION,-4); // $v1 = substr($data['compatibility']['database'],-4);
if($v2 > $v1) { // $v2 = substr(DB_UPDATE_VERSION,-4);
$t = sprintf( t('Warning: Database versions differ by %1$d updates.'), $v2 - $v1 ); // if($v2 > $v1) {
notice($t); // $t = sprintf( t('Warning: Database versions differ by %1$d updates.'), $v2 - $v1 );
} // notice($t);
// }
} //
// }
if($moving) if($moving)
$seize = 1; $seize = 1;

View File

@ -74,7 +74,7 @@
{{include file="field_select.tpl" field=$access_policy}} {{include file="field_select.tpl" field=$access_policy}}
{{include file="field_input.tpl" field=$location}} {{include file="field_input.tpl" field=$location}}
{{include file="field_input.tpl" field=$sellpage}} {{include file="field_input.tpl" field=$sellpage}}
{{include file="field_input.tpl" field=$firstpage}} {{include file="field_input.tpl" field=$first_page}}
<div class="submit"><input type="submit" name="page_site" value="{{$submit}}" /></div> <div class="submit"><input type="submit" name="page_site" value="{{$submit}}" /></div>