Replace "Probe::isProbable" with "Protocol::supportsProbe"
This commit is contained in:
		
					parent
					
						
							
								a52c1cde9c
							
						
					
				
			
			
				commit
				
					
						2e46d64ea0
					
				
			
		
					 2 changed files with 10 additions and 5 deletions
				
			
		| 
						 | 
				
			
			@ -315,7 +315,12 @@ class Protocol
 | 
			
		|||
	 */
 | 
			
		||||
	public static function supportsProbe($protocol): bool
 | 
			
		||||
	{
 | 
			
		||||
		if (in_array($protocol, self::NATIVE_SUPPORT)) {
 | 
			
		||||
		// "Mail" can only be probed for a specific user in a specific condition, so we are ignoring it here.
 | 
			
		||||
		if ($protocol == self::MAIL) {
 | 
			
		||||
			return false;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		if (in_array($protocol, array_merge(self::NATIVE_SUPPORT, [self::ZOT, self::PHANTOM]))) {
 | 
			
		||||
			return true;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -361,7 +361,7 @@ class Contact
 | 
			
		|||
		$background_update = DI::config()->get('system', 'update_active_contacts') ? $contact['local-data'] : true;
 | 
			
		||||
 | 
			
		||||
		// Update the contact in the background if needed
 | 
			
		||||
		if ($background_update && !self::isLocal($url) && Probe::isProbable($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
		if ($background_update && !self::isLocal($url) && Protocol::supportsProbe($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
			try {
 | 
			
		||||
				UpdateContact::add(['priority' => Worker::PRIORITY_LOW, 'dont_fork' => true], $contact['id']);
 | 
			
		||||
			} catch (\InvalidArgumentException $e) {
 | 
			
		||||
| 
						 | 
				
			
			@ -1279,7 +1279,7 @@ class Contact
 | 
			
		|||
 | 
			
		||||
			$background_update = DI::config()->get('system', 'update_active_contacts') ? $contact['local-data'] : true;
 | 
			
		||||
 | 
			
		||||
			if ($background_update && !self::isLocal($url) && Probe::isProbable($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
			if ($background_update && !self::isLocal($url) && Protocol::supportsProbe($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
				try {
 | 
			
		||||
					UpdateContact::add(['priority' => Worker::PRIORITY_LOW, 'dont_fork' => true], $contact['id']);
 | 
			
		||||
				} catch (\InvalidArgumentException $e) {
 | 
			
		||||
| 
						 | 
				
			
			@ -2704,7 +2704,7 @@ class Contact
 | 
			
		|||
 | 
			
		||||
		$updated = DateTimeFormat::utcNow();
 | 
			
		||||
 | 
			
		||||
		if (!Probe::isProbable($ret['network']) && !Probe::isProbable($contact['network'])) {
 | 
			
		||||
		if (!Protocol::supportsProbe($ret['network']) && !Protocol::supportsProbe($contact['network'])) {
 | 
			
		||||
			// Periodical checks are only done on federated contacts
 | 
			
		||||
			$failed_next_update  = null;
 | 
			
		||||
			$success_next_update = null;
 | 
			
		||||
| 
						 | 
				
			
			@ -3596,7 +3596,7 @@ class Contact
 | 
			
		|||
			if (empty($contact['id']) && Network::isValidHttpUrl($url)) {
 | 
			
		||||
				Worker::add(Worker::PRIORITY_LOW, 'AddContact', 0, $url);
 | 
			
		||||
				++$added;
 | 
			
		||||
			} elseif (!empty($contact['network']) && Probe::isProbable($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
			} elseif (!empty($contact['network']) && Protocol::supportsProbe($contact['network']) && ($contact['next-update'] < DateTimeFormat::utcNow())) {
 | 
			
		||||
				try {
 | 
			
		||||
					UpdateContact::add(['priority' => Worker::PRIORITY_LOW, 'dont_fork' => true], $contact['id']);
 | 
			
		||||
					++$updated;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue