浏览代码

Update Discover apis

Daniel Supernault 2 年之前
父节点
当前提交
243fcac7d5
共有 2 个文件被更改,包括 22 次插入16 次删除
  1. 20 16
      app/Http/Controllers/DiscoverController.php
  2. 2 0
      routes/api.php

+ 20 - 16
app/Http/Controllers/DiscoverController.php

@@ -123,7 +123,7 @@ class DiscoverController extends Controller
 
 
 	public function trendingApi(Request $request)
 	public function trendingApi(Request $request)
 	{
 	{
-		abort_if(config('instance.discover.public') == false && !Auth::check(), 403);
+		abort_if(config('instance.discover.public') == false && !$request->user(), 403);
 
 
 		$this->validate($request, [
 		$this->validate($request, [
 			'range' => 'nullable|string|in:daily,monthly,yearly',
 			'range' => 'nullable|string|in:daily,monthly,yearly',
@@ -179,21 +179,25 @@ class DiscoverController extends Controller
 
 
 	public function trendingHashtags(Request $request)
 	public function trendingHashtags(Request $request)
 	{
 	{
-		$res = StatusHashtag::select('hashtag_id', \DB::raw('count(*) as total'))
-			->groupBy('hashtag_id')
-			->orderBy('total','desc')
-			->where('created_at', '>', now()->subDays(90))
-			->take(9)
-			->get()
-			->map(function($h) {
-				$hashtag = $h->hashtag;
-				return [
-					'id' => $hashtag->id,
-					'total' => $h->total,
-					'name' => '#'.$hashtag->name,
-					'url' => $hashtag->url('?src=dsh1')
-				];
-			});
+		abort_if(!$request->user(), 403);
+
+		$res = Cache::remember('api:discover:v1.1:trending:hashtags', 3600, function() {
+			return StatusHashtag::select('hashtag_id', \DB::raw('count(*) as total'))
+				->groupBy('hashtag_id')
+				->orderBy('total','desc')
+				->where('created_at', '>', now()->subDays(90))
+				->take(9)
+				->get()
+				->map(function($h) {
+					$hashtag = $h->hashtag;
+					return [
+						'id' => $hashtag->id,
+						'total' => $h->total,
+						'name' => '#'.$hashtag->name,
+						'url' => $hashtag->url()
+					];
+				});
+		});
 		return $res;
 		return $res;
 	}
 	}
 
 

+ 2 - 0
routes/api.php

@@ -112,6 +112,8 @@ Route::group(['prefix' => 'api'], function() use($middleware) {
 
 
 		Route::group(['prefix' => 'discover'], function () use($middleware) {
 		Route::group(['prefix' => 'discover'], function () use($middleware) {
 			Route::get('accounts/popular', 'Api\ApiV1Controller@discoverAccountsPopular')->middleware($middleware);
 			Route::get('accounts/popular', 'Api\ApiV1Controller@discoverAccountsPopular')->middleware($middleware);
+			Route::get('posts/trending', 'DiscoverController@trendingApi')->middleware($middleware);
+			Route::get('posts/hashtags', 'DiscoverController@trendingHashtags')->middleware($middleware);
 		});
 		});
 	});
 	});