Consolidate time periods methods into base measure (#30757)
This commit is contained in:
		
							parent
							
								
									80743e0e7f
								
							
						
					
					
						commit
						a8b3d2e4e4
					
				| @ -22,12 +22,4 @@ class Admin::Metrics::Measure::ActiveUsersMeasure < Admin::Metrics::Measure::Bas | ||||
|   def activity_tracker | ||||
|     @activity_tracker ||= ActivityTracker.new('activity:logins', :unique) | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| end | ||||
|  | ||||
| @ -86,11 +86,11 @@ class Admin::Metrics::Measure::BaseMeasure | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at..@end_at) | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at - length_of_period)..(@end_at - length_of_period)) | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def length_of_period | ||||
|  | ||||
| @ -48,14 +48,6 @@ class Admin::Metrics::Measure::InstanceAccountsMeasure < Admin::Metrics::Measure | ||||
|     SQL | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -49,14 +49,6 @@ class Admin::Metrics::Measure::InstanceFollowersMeasure < Admin::Metrics::Measur | ||||
|     SQL | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -49,14 +49,6 @@ class Admin::Metrics::Measure::InstanceFollowsMeasure < Admin::Metrics::Measure: | ||||
|     SQL | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -58,14 +58,6 @@ class Admin::Metrics::Measure::InstanceMediaAttachmentsMeasure < Admin::Metrics: | ||||
|     SQL | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -49,14 +49,6 @@ class Admin::Metrics::Measure::InstanceReportsMeasure < Admin::Metrics::Measure: | ||||
|     SQL | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -58,14 +58,6 @@ class Admin::Metrics::Measure::InstanceStatusesMeasure < Admin::Metrics::Measure | ||||
|     Mastodon::Snowflake.id_at(@end_at.end_of_day, with_random: false) | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:domain, :include_subdomains) | ||||
|   end | ||||
|  | ||||
| @ -22,12 +22,4 @@ class Admin::Metrics::Measure::InteractionsMeasure < Admin::Metrics::Measure::Ba | ||||
|   def activity_tracker | ||||
|     @activity_tracker ||= ActivityTracker.new('activity:interactions', :basic) | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| end | ||||
|  | ||||
| @ -27,14 +27,6 @@ class Admin::Metrics::Measure::TagAccountsMeasure < Admin::Metrics::Measure::Bas | ||||
|     @tag ||= Tag.find(params[:id]) | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:id) | ||||
|   end | ||||
|  | ||||
| @ -27,14 +27,6 @@ class Admin::Metrics::Measure::TagUsesMeasure < Admin::Metrics::Measure::BaseMea | ||||
|     @tag ||= Tag.find(params[:id]) | ||||
|   end | ||||
| 
 | ||||
|   def time_period | ||||
|     (@start_at.to_date..@end_at.to_date) | ||||
|   end | ||||
| 
 | ||||
|   def previous_time_period | ||||
|     ((@start_at.to_date - length_of_period)..(@end_at.to_date - length_of_period)) | ||||
|   end | ||||
| 
 | ||||
|   def params | ||||
|     @params.permit(:id) | ||||
|   end | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user