Fixes: (#5477)
- use DBA::isResult() and not count() - fixed 4 E_NOTICE when POST fields are not set
This commit is contained in:
		
					parent
					
						
							
								14a4d5eb59
							
						
					
				
			
			
				commit
				
					
						00595de6a3
					
				
			
		
					 1 changed files with 6 additions and 5 deletions
				
			
		| 
						 | 
				
			
			@ -124,7 +124,8 @@ function events_post(App $a) {
 | 
			
		|||
	$c = q("SELECT `id` FROM `contact` WHERE `uid` = %d AND `self` LIMIT 1",
 | 
			
		||||
		intval(local_user())
 | 
			
		||||
	);
 | 
			
		||||
	if (count($c)) {
 | 
			
		||||
 | 
			
		||||
	if (DBA::isResult($c)) {
 | 
			
		||||
		$self = $c[0]['id'];
 | 
			
		||||
	} else {
 | 
			
		||||
		$self = 0;
 | 
			
		||||
| 
						 | 
				
			
			@ -132,10 +133,10 @@ function events_post(App $a) {
 | 
			
		|||
 | 
			
		||||
 | 
			
		||||
	if ($share) {
 | 
			
		||||
		$str_group_allow   = perms2str($_POST['group_allow']);
 | 
			
		||||
		$str_contact_allow = perms2str($_POST['contact_allow']);
 | 
			
		||||
		$str_group_deny    = perms2str($_POST['group_deny']);
 | 
			
		||||
		$str_contact_deny  = perms2str($_POST['contact_deny']);
 | 
			
		||||
		$str_group_allow   = !empty($_POST['group_allow'])   ? perms2str($_POST['group_allow'])   : '';
 | 
			
		||||
		$str_contact_allow = !empty($_POST['contact_allow']) ? perms2str($_POST['contact_allow']) : '';
 | 
			
		||||
		$str_group_deny    = !empty($_POST['group_deny'])    ? perms2str($_POST['group_deny'])    : '';
 | 
			
		||||
		$str_contact_deny  = !empty($_POST['contact_deny'])  ? perms2str($_POST['contact_deny'])  : '';
 | 
			
		||||
 | 
			
		||||
		// Undo the pseudo-contact of self, since there are real contacts now
 | 
			
		||||
		if (strpos($str_contact_allow, '<' . $self . '>') !== false ) {
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue