Raise an error on getting activity uri for remote status (#4984)
We had returned `nil` for that case, but this raises an error instead, as a wrong usage of the method. This method is currently only used in ActivitySerializer.
This commit is contained in:
		@@ -37,7 +37,7 @@ class ActivityPub::TagManager
 | 
				
			|||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def activity_uri_for(target)
 | 
					  def activity_uri_for(target)
 | 
				
			||||||
    return nil unless %i(note comment activity).include?(target.object_type) && target.local?
 | 
					    raise ArgumentError, 'target must be a local activity' unless %i(note comment activity).include?(target.object_type) && target.local?
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    activity_account_status_url(target.account, target)
 | 
					    activity_account_status_url(target.account, target)
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user