From 6679214808f741cd373ed5ee80e37f2666c3a8e4 Mon Sep 17 00:00:00 2001 From: koogua Date: Sat, 26 Sep 2020 18:59:02 +0800 Subject: [PATCH] =?UTF-8?q?!1=20=E7=B2=BE=E7=AE=80=E4=BC=98=E5=8C=96?= =?UTF-8?q?=E4=BB=A3=E7=A0=81=20*=20=E4=BC=98=E5=8C=96=E4=BB=A3=E7=A0=81?= =?UTF-8?q?=20*=20=E9=87=8D=E6=96=B0=E6=8F=90=E4=BA=A4LICENSE=20*=20?= =?UTF-8?q?=E6=B5=8B=E8=AF=95=E5=89=8D=E5=8F=B0=E6=94=AF=E4=BB=98=EF=BC=8C?= =?UTF-8?q?=E5=A2=9E=E5=8A=A0=E8=8B=A5=E5=B9=B2=E5=91=BD=E4=BB=A4=E8=A1=8C?= =?UTF-8?q?=E5=BF=AB=E6=8D=B7=E6=96=B9=E6=B3=95=20*=20=E5=88=A0=E9=99=A4?= =?UTF-8?q?=E6=96=87=E4=BB=B6=20LICENSE?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 4 +-- app/Console/Tasks/SyncCourseIndexTask.php | 2 -- app/Console/Tasks/SyncGroupIndexTask.php | 2 -- app/Console/Tasks/SyncLearningTask.php | 2 -- app/Console/Tasks/SyncUserIndexTask.php | 2 -- app/Console/Tasks/UpgradeTask.php | 4 +-- app/Http/Admin/Services/WxpayTest.php | 13 +++++-- app/Http/Admin/Views/course/add.volt | 2 +- app/Http/Admin/Views/course/list.volt | 2 +- app/Http/Admin/Views/course/search.volt | 2 +- app/Http/Admin/Views/setting/pay_wxpay.volt | 2 +- app/Http/Home/Controllers/Controller.php | 18 +++++----- app/Http/Home/Controllers/OrderController.php | 5 ++- app/Http/Home/Services/Trade.php | 13 ++++++- app/Http/Home/Views/order/info.volt | 4 +-- app/Http/Home/Views/order/pay.volt | 9 ++--- app/Http/Home/Views/partials/header.volt | 2 +- .../Home/Views/teacher/console/consults.volt | 2 +- .../Home/Views/teacher/console/courses.volt | 2 +- .../Home/Views/teacher/console/lives.volt | 2 +- app/Http/Home/Views/user/console/account.volt | 2 +- .../Home/Views/user/console/consults.volt | 2 +- app/Http/Home/Views/user/console/courses.volt | 2 +- .../Home/Views/user/console/favorites.volt | 2 +- app/Http/Home/Views/user/console/friends.volt | 2 +- app/Http/Home/Views/user/console/groups.volt | 2 +- app/Http/Home/Views/user/console/orders.volt | 2 +- app/Http/Home/Views/user/console/profile.volt | 2 +- app/Http/Home/Views/user/console/refunds.volt | 2 +- app/Http/Home/Views/user/console/reviews.volt | 2 +- app/Models/Audit.php | 5 +++ app/Models/Course.php | 2 +- app/Providers/Cookie.php | 7 ++++ app/Providers/Escaper.php | 4 +-- app/Services/Pay/Alipay.php | 1 + app/Validators/Category.php | 28 ++++++++------- app/Validators/Chapter.php | 34 ++++++++++++------- app/Validators/Course.php | 28 ++++++++------- app/Validators/Help.php | 26 +++++++------- app/Validators/ImGroup.php | 28 ++++++++------- app/Validators/Package.php | 28 ++++++++------- app/Validators/Page.php | 28 ++++++++------- app/Validators/Topic.php | 28 ++++++++------- app/Validators/User.php | 28 ++++++++------- public/static/home/css/common.css | 23 ++----------- public/static/home/js/order.pay.js | 5 ++- 46 files changed, 237 insertions(+), 180 deletions(-) diff --git a/README.md b/README.md index 22a6034c..abfadbd8 100644 --- a/README.md +++ b/README.md @@ -29,7 +29,7 @@ #### 使用协议 -虽然尝试了解过开源协议，但是理解的模棱两可，干脆用自己的协议吧，算不算开源我也不很清楚。 +虽然尝试了解过开源协议，但是理解的模棱两可，干脆用自己的协议吧。 1. 本系统属于强业务类型，非通用类库框架，不适合再次衍生发布。 2. 在保留我们版权标识的前提下，用户可以修改以满足自己的需求，可以用于商业用途。 @@ -74,7 +74,7 @@ #### 有商业服务吗？ -生存是一个问题，生存才能发展，我们为用户提供的服务包括： +生存才能发展，我们目前提供的服务包括： - 系统安装 - 系统定制 diff --git a/app/Console/Tasks/SyncCourseIndexTask.php b/app/Console/Tasks/SyncCourseIndexTask.php index 1578f036..ae1f7076 100644 --- a/app/Console/Tasks/SyncCourseIndexTask.php +++ b/app/Console/Tasks/SyncCourseIndexTask.php @@ -12,8 +12,6 @@ class SyncCourseIndexTask extends Task public function mainAction() { - $cache = $this->getCache(); - $redis = $this->getRedis(); $key = $this->getSyncKey(); diff --git a/app/Console/Tasks/SyncGroupIndexTask.php b/app/Console/Tasks/SyncGroupIndexTask.php index d48223de..a233c1a5 100644 --- a/app/Console/Tasks/SyncGroupIndexTask.php +++ b/app/Console/Tasks/SyncGroupIndexTask.php @@ -12,8 +12,6 @@ class SyncGroupIndexTask extends Task public function mainAction() { - $cache = $this->getCache(); - $redis = $this->getRedis(); $key = $this->getSyncKey(); diff --git a/app/Console/Tasks/SyncLearningTask.php b/app/Console/Tasks/SyncLearningTask.php index 8adbc36e..af1f41aa 100644 --- a/app/Console/Tasks/SyncLearningTask.php +++ b/app/Console/Tasks/SyncLearningTask.php @@ -16,8 +16,6 @@ class SyncLearningTask extends Task public function mainAction() { - $cache = $this->getCache(); - $redis = $this->getRedis(); $sync = new LearningSync(); diff --git a/app/Console/Tasks/SyncUserIndexTask.php b/app/Console/Tasks/SyncUserIndexTask.php index 7e55f3da..f792a465 100644 --- a/app/Console/Tasks/SyncUserIndexTask.php +++ b/app/Console/Tasks/SyncUserIndexTask.php @@ -12,8 +12,6 @@ class SyncUserIndexTask extends Task public function mainAction() { - $cache = $this->getCache(); - $redis = $this->getRedis(); $key = $this->getSyncKey(); diff --git a/app/Console/Tasks/UpgradeTask.php b/app/Console/Tasks/UpgradeTask.php index a8600332..13bb53fb 100644 --- a/app/Console/Tasks/UpgradeTask.php +++ b/app/Console/Tasks/UpgradeTask.php @@ -43,7 +43,7 @@ class UpgradeTask extends Task public function resetAnnotationAction() { $config = $this->getConfig(); - $cache = $this->getCache(); + $redis = $this->getRedis(); $dbIndex = $config->path('annotation.db'); @@ -74,7 +74,7 @@ class UpgradeTask extends Task public function resetMetadataAction() { $config = $this->getConfig(); - $cache = $this->getCache(); + $redis = $this->getRedis(); $dbIndex = $config->path('metadata.db'); diff --git a/app/Http/Admin/Services/WxpayTest.php b/app/Http/Admin/Services/WxpayTest.php index b6094d6e..053d3e1f 100644 --- a/app/Http/Admin/Services/WxpayTest.php +++ b/app/Http/Admin/Services/WxpayTest.php @@ -14,9 +14,18 @@ class WxpayTest extends PayTest { $wxpayService = new WxpayService(); - $qrcode = $wxpayService->scan($trade); + $code = $wxpayService->scan($trade); - return $qrcode ?: false; + $codeUrl = null; + + if ($code) { + $codeUrl = $this->url->get( + ['for' => 'home.qrcode'], + ['text' => urlencode($code)] + ); + } + + return $codeUrl ?: false; } public function status($tradeNo) diff --git a/app/Http/Admin/Views/course/add.volt b/app/Http/Admin/Views/course/add.volt index 66a731bf..a90cd2e7 100644 --- a/app/Http/Admin/Views/course/add.volt +++ b/app/Http/Admin/Views/course/add.volt @@ -11,7 +11,7 @@
- +
diff --git a/app/Http/Admin/Views/course/list.volt b/app/Http/Admin/Views/course/list.volt index 9580510d..450a3641 100644 --- a/app/Http/Admin/Views/course/list.volt +++ b/app/Http/Admin/Views/course/list.volt @@ -8,7 +8,7 @@ {% elseif value == 2 %} 直播 {% elseif value == 3 %} - 图文 + 专栏 {% endif %} {%- endmacro %} diff --git a/app/Http/Admin/Views/course/search.volt b/app/Http/Admin/Views/course/search.volt index 06be131d..9a8d0935 100644 --- a/app/Http/Admin/Views/course/search.volt +++ b/app/Http/Admin/Views/course/search.volt @@ -35,7 +35,7 @@
- +
diff --git a/app/Http/Admin/Views/setting/pay_wxpay.volt b/app/Http/Admin/Views/setting/pay_wxpay.volt index 14e7bf66..beb2c4f9 100644 --- a/app/Http/Admin/Views/setting/pay_wxpay.volt +++ b/app/Http/Admin/Views/setting/pay_wxpay.volt @@ -19,7 +19,7 @@
- +
diff --git a/app/Http/Home/Controllers/Controller.php b/app/Http/Home/Controllers/Controller.php index 78b8e630..30819779 100644 --- a/app/Http/Home/Controllers/Controller.php +++ b/app/Http/Home/Controllers/Controller.php @@ -3,7 +3,6 @@ namespace App\Http\Home\Controllers; use App\Caches\NavTreeList as NavCache; -use App\Caches\Setting as SettingCache; use App\Library\AppInfo as AppInfo; use App\Library\Seo as Seo; use App\Models\User as UserModel; @@ -116,9 +115,7 @@ class Controller extends \Phalcon\Mvc\Controller protected function getSiteInfo() { - $appService = new AppService(); - - return $appService->getSettings('site'); + return $this->getSettings('site'); } protected function getAppInfo() @@ -128,8 +125,6 @@ class Controller extends \Phalcon\Mvc\Controller protected function getImInfo() { - $cache = new SettingCache(); - $websocket = $this->getConfig()->get('websocket'); /** @@ -142,8 +137,8 @@ class Controller extends \Phalcon\Mvc\Controller } return [ - 'main' => $cache->get('im.main'), - 'cs' => $cache->get('im.cs'), + 'main' => $this->getSettings('im.main'), + 'cs' => $this->getSettings('im.cs'), 'ws' => $websocket, ]; } @@ -155,4 +150,11 @@ class Controller extends \Phalcon\Mvc\Controller return $appService->getConfig(); } + protected function getSettings($section) + { + $appService = new AppService(); + + return $appService->getSettings($section); + } + } diff --git a/app/Http/Home/Controllers/OrderController.php b/app/Http/Home/Controllers/OrderController.php index 9f4550cb..591c8187 100644 --- a/app/Http/Home/Controllers/OrderController.php +++ b/app/Http/Home/Controllers/OrderController.php @@ -67,7 +67,10 @@ class OrderController extends Controller $order = $service->handle(); - $location = $this->url->get(['for' => 'home.order.pay'], ['sn' => $order->sn]); + $location = $this->url->get( + ['for' => 'home.order.pay'], + ['sn' => $order->sn] + ); return $this->jsonSuccess(['location' => $location]); } diff --git a/app/Http/Home/Services/Trade.php b/app/Http/Home/Services/Trade.php index 50e176dd..629fc986 100644 --- a/app/Http/Home/Services/Trade.php +++ b/app/Http/Home/Services/Trade.php @@ -72,9 +72,20 @@ class Trade extends Service protected function getWxpayQrCode(TradeModel $trade) { + $qrCode = null; + $service = new WxpayService(); - return $service->scan($trade); + $text = $service->scan($trade); + + if ($text) { + $qrCode = $this->url->get( + ['for' => 'home.qrcode'], + ['text' => urlencode($text)] + ); + } + + return $qrCode; } } diff --git a/app/Http/Home/Views/order/info.volt b/app/Http/Home/Views/order/info.volt index 33b78cd5..c4472b16 100644 --- a/app/Http/Home/Views/order/info.volt +++ b/app/Http/Home/Views/order/info.volt @@ -21,10 +21,10 @@
- {% if order.status == 'pending' %} + {% if order.status == 1 %} 立即支付 {% endif %} - {% if (order.item_type in ['course','package']) and (order.status == 'finished') %} + {% if (order.item_type in [1,2]) and (order.status == 3) %} 申请退款 {% endif %}
diff --git a/app/Http/Home/Views/order/pay.volt b/app/Http/Home/Views/order/pay.volt index 5b0cf144..16bca937 100644 --- a/app/Http/Home/Views/order/pay.volt +++ b/app/Http/Home/Views/order/pay.volt @@ -2,6 +2,9 @@ {% block content %} + {% set create_url = url({'for':'home.trade.create'}) %} + {% set status_url = url({'for':'home.trade.status'}) %} +
首页 支付订单 @@ -14,10 +17,8 @@ 支付金额：{{ '￥%0.2f'|format(order.amount) }}
- {% set create_url = url({'for':'home.trade.create'}) %} - {% set status_url = url({'for':'home.trade.status'}) %} - - + {{ image('home/img/alipay.png') }} + {{ image('home/img/wxpay.png') }}
友情提示：请在12小时内完成支付，有问题请联系客服 diff --git a/app/Http/Home/Views/partials/header.volt b/app/Http/Home/Views/partials/header.volt index 12376d7b..fbb2232b 100644 --- a/app/Http/Home/Views/partials/header.volt +++ b/app/Http/Home/Views/partials/header.volt @@ -27,7 +27,7 @@ 搜索
  • - 会员 + 会员
  • 微聊 diff --git a/app/Http/Home/Views/teacher/console/consults.volt b/app/Http/Home/Views/teacher/console/consults.volt index 6ba376f9..39349f99 100644 --- a/app/Http/Home/Views/teacher/console/consults.volt +++ b/app/Http/Home/Views/teacher/console/consults.volt @@ -7,7 +7,7 @@ {% set status_types = {'all':'全部','pending':'待回复','replied':'已回复'} %} {% set status = request.get('status','trim','all') %} -
    +
    {{ partial('teacher/console/menu') }}
    diff --git a/app/Http/Home/Views/teacher/console/courses.volt b/app/Http/Home/Views/teacher/console/courses.volt index 61d1f3fe..21d10441 100644 --- a/app/Http/Home/Views/teacher/console/courses.volt +++ b/app/Http/Home/Views/teacher/console/courses.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('teacher/console/menu') }}
    diff --git a/app/Http/Home/Views/teacher/console/lives.volt b/app/Http/Home/Views/teacher/console/lives.volt index efd62921..f247765d 100644 --- a/app/Http/Home/Views/teacher/console/lives.volt +++ b/app/Http/Home/Views/teacher/console/lives.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('teacher/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/account.volt b/app/Http/Home/Views/user/console/account.volt index 61ca83ff..882c5547 100644 --- a/app/Http/Home/Views/user/console/account.volt +++ b/app/Http/Home/Views/user/console/account.volt @@ -6,7 +6,7 @@ {% set edit_phone_url = url({'for':'home.account.edit_phone'}) %} {% set edit_email_url = url({'for':'home.account.edit_email'}) %} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/consults.volt b/app/Http/Home/Views/user/console/consults.volt index 45c50756..f7284323 100644 --- a/app/Http/Home/Views/user/console/consults.volt +++ b/app/Http/Home/Views/user/console/consults.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/courses.volt b/app/Http/Home/Views/user/console/courses.volt index 962a7f1a..7d9b0f70 100644 --- a/app/Http/Home/Views/user/console/courses.volt +++ b/app/Http/Home/Views/user/console/courses.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/favorites.volt b/app/Http/Home/Views/user/console/favorites.volt index 536ac712..768f9506 100644 --- a/app/Http/Home/Views/user/console/favorites.volt +++ b/app/Http/Home/Views/user/console/favorites.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/friends.volt b/app/Http/Home/Views/user/console/friends.volt index 79efd4a6..0a45e4b6 100644 --- a/app/Http/Home/Views/user/console/friends.volt +++ b/app/Http/Home/Views/user/console/friends.volt @@ -4,7 +4,7 @@ {{ partial('macros/user') }} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/groups.volt b/app/Http/Home/Views/user/console/groups.volt index e01eb555..7716a31c 100644 --- a/app/Http/Home/Views/user/console/groups.volt +++ b/app/Http/Home/Views/user/console/groups.volt @@ -9,7 +9,7 @@ {% set joined_class = scope == 'joined' ? 'layui-btn layui-btn-xs' : 'none' %} {% set owned_class = scope == 'owned' ? 'layui-btn layui-btn-xs' : 'none' %} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/orders.volt b/app/Http/Home/Views/user/console/orders.volt index 78fc09d1..324139e3 100644 --- a/app/Http/Home/Views/user/console/orders.volt +++ b/app/Http/Home/Views/user/console/orders.volt @@ -7,7 +7,7 @@ {% set status_types = {'0':'全部','1':'待支付','3':'已完成','4':'已关闭','5':'已退款'} %} {% set status = request.get('status','trim','0') %} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/profile.volt b/app/Http/Home/Views/user/console/profile.volt index f52f8ab7..cbf2a6c8 100644 --- a/app/Http/Home/Views/user/console/profile.volt +++ b/app/Http/Home/Views/user/console/profile.volt @@ -7,7 +7,7 @@ {% set gender_female_checked = user.gender == 2 ? 'checked' : '' %} {% set gender_none_checked = user.gender == 3 ? 'checked' : '' %} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/refunds.volt b/app/Http/Home/Views/user/console/refunds.volt index b765c0a6..78c3be06 100644 --- a/app/Http/Home/Views/user/console/refunds.volt +++ b/app/Http/Home/Views/user/console/refunds.volt @@ -7,7 +7,7 @@ {% set status_types = {'0':'全部','1':'待处理','2':'已取消','3':'退款中','5':'已完成'} %} {% set status = request.get('status','int','0') %} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Http/Home/Views/user/console/reviews.volt b/app/Http/Home/Views/user/console/reviews.volt index a0d40c46..7634299e 100644 --- a/app/Http/Home/Views/user/console/reviews.volt +++ b/app/Http/Home/Views/user/console/reviews.volt @@ -4,7 +4,7 @@ {{ partial('macros/course') }} -
    +
    {{ partial('user/console/menu') }}
    diff --git a/app/Models/Audit.php b/app/Models/Audit.php index 273717e8..bd1dffd1 100644 --- a/app/Models/Audit.php +++ b/app/Models/Audit.php @@ -71,6 +71,11 @@ class Audit extends Model $this->create_time = time(); if (is_array($this->req_data) && !empty($this->req_data)) { + foreach ($this->req_data as $key => $value) { + if (kg_strlen($value) > 255) { + $this->req_data[$key] = kg_substr($value, 0, 255); + } + } $this->req_data = kg_json_encode($this->req_data); } else { $this->req_data = ''; diff --git a/app/Models/Course.php b/app/Models/Course.php index 00caad46..4e995e9e 100644 --- a/app/Models/Course.php +++ b/app/Models/Course.php @@ -367,7 +367,7 @@ class Course extends Model return [ self::MODEL_VOD => '点播', self::MODEL_LIVE => '直播', - self::MODEL_READ => '图文', + self::MODEL_READ => '专栏', ]; } diff --git a/app/Providers/Cookie.php b/app/Providers/Cookie.php index 9609c3ce..d4b89cc0 100644 --- a/app/Providers/Cookie.php +++ b/app/Providers/Cookie.php @@ -2,6 +2,8 @@ namespace App\Providers; +use Phalcon\Http\Response\Cookies as PhCookies; + class Cookie extends Provider { @@ -11,6 +13,11 @@ class Cookie extends Provider { $this->di->setShared($this->serviceName, function () { + $cookies = new PhCookies(); + + $cookies->useEncryption(true); + + return $cookies; }); } diff --git a/app/Providers/Escaper.php b/app/Providers/Escaper.php index 5505f214..2bb95d7a 100644 --- a/app/Providers/Escaper.php +++ b/app/Providers/Escaper.php @@ -2,7 +2,7 @@ namespace App\Providers; -use Phalcon\Escaper as PhalconEscaper; +use Phalcon\Escaper as PhEscaper; class Escaper extends Provider { @@ -11,7 +11,7 @@ class Escaper extends Provider public function register() { - $this->di->setShared($this->serviceName, PhalconEscaper::class); + $this->di->setShared($this->serviceName, PhEscaper::class); } } \ No newline at end of file diff --git a/app/Services/Pay/Alipay.php b/app/Services/Pay/Alipay.php index 14fad063..9413f139 100644 --- a/app/Services/Pay/Alipay.php +++ b/app/Services/Pay/Alipay.php @@ -87,6 +87,7 @@ class Alipay extends PayService /** * 异步通知 + * * @return Response|bool */ public function notify() diff --git a/app/Validators/Category.php b/app/Validators/Category.php index ead79b30..98be43ad 100644 --- a/app/Validators/Category.php +++ b/app/Validators/Category.php @@ -18,18 +18,7 @@ class Category extends Validator */ public function checkCategoryCache($id) { - $id = intval($id); - - $maxCategoryIdCache = new MaxCategoryIdCache(); - - $maxCategoryId = $maxCategoryIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxCategoryId) { - throw new BadRequestException('category.not_found'); - } + $this->checkId($id); $categoryCache = new CategoryCache(); @@ -44,6 +33,8 @@ class Category extends Validator public function checkCategory($id) { + $this->checkId($id); + $categoryRepo = new CategoryRepo(); $category = $categoryRepo->findById($id); @@ -55,6 +46,19 @@ class Category extends Validator return $category; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxCategoryIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('category.not_found'); + } + } + public function checkParent($id) { $categoryRepo = new CategoryRepo(); diff --git a/app/Validators/Chapter.php b/app/Validators/Chapter.php index 788084e7..1517590a 100644 --- a/app/Validators/Chapter.php +++ b/app/Validators/Chapter.php @@ -19,18 +19,7 @@ class Chapter extends Validator */ public function checkChapterCache($id) { - $id = intval($id); - - $maxChapterIdCache = new MaxChapterIdCache(); - - $maxChapterId = $maxChapterIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxChapterId) { - throw new BadRequestException('chapter.not_found'); - } + $this->checkId($id); $chapterCache = new ChapterCache(); @@ -45,6 +34,8 @@ class Chapter extends Validator public function checkChapterVod($id) { + $this->checkId($id); + $chapterRepo = new ChapterRepo(); $chapterVod = $chapterRepo->findChapterVod($id); @@ -58,6 +49,8 @@ class Chapter extends Validator public function checkChapterLive($id) { + $this->checkId($id); + $chapterRepo = new ChapterRepo(); $chapterLive = $chapterRepo->findChapterLive($id); @@ -71,6 +64,8 @@ class Chapter extends Validator public function checkChapterRead($id) { + $this->checkId($id); + $chapterRepo = new ChapterRepo(); $chapterRead = $chapterRepo->findChapterRead($id); @@ -84,6 +79,8 @@ class Chapter extends Validator public function checkChapter($id) { + $this->checkId($id); + $chapterRepo = new ChapterRepo(); $chapter = $chapterRepo->findById($id); @@ -95,6 +92,19 @@ class Chapter extends Validator return $chapter; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxChapterIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('chapter.not_found'); + } + } + public function checkCourse($id) { $validator = new Course(); diff --git a/app/Validators/Course.php b/app/Validators/Course.php index 14424f78..3bafb387 100644 --- a/app/Validators/Course.php +++ b/app/Validators/Course.php @@ -19,18 +19,7 @@ class Course extends Validator */ public function checkCourseCache($id) { - $id = intval($id); - - $maxCache = new MaxCourseIdCache(); - - $maxId = $maxCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxId) { - throw new BadRequestException('course.not_found'); - } + $this->checkId($id); $courseCache = new CourseCache(); @@ -45,6 +34,8 @@ class Course extends Validator public function checkCourse($id) { + $this->checkId($id); + $courseRepo = new CourseRepo(); $course = $courseRepo->findById($id); @@ -56,6 +47,19 @@ class Course extends Validator return $course; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxCourseIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('course.not_found'); + } + } + public function checkModel($model) { $list = CourseModel::modelTypes(); diff --git a/app/Validators/Help.php b/app/Validators/Help.php index babab1b5..e9da769d 100644 --- a/app/Validators/Help.php +++ b/app/Validators/Help.php @@ -18,18 +18,7 @@ class Help extends Validator */ public function checkHelpCache($id) { - $id = intval($id); - - $maxHelpIdCache = new MaxHelpIdCache(); - - $maxHelpId = $maxHelpIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxHelpId) { - throw new BadRequestException('help.not_found'); - } + $this->checkId($id); $helpCache = new HelpCache(); @@ -55,6 +44,19 @@ class Help extends Validator return $help; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxHelpIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('help.not_found'); + } + } + public function checkCategory($id) { $validator = new Category(); diff --git a/app/Validators/ImGroup.php b/app/Validators/ImGroup.php index 5e2a2c38..80c042f5 100644 --- a/app/Validators/ImGroup.php +++ b/app/Validators/ImGroup.php @@ -19,18 +19,7 @@ class ImGroup extends Validator */ public function checkGroupCache($id) { - $id = intval($id); - - $maxGroupIdCache = new MaxImGroupIdCache(); - - $maxGroupId = $maxGroupIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxGroupId) { - throw new BadRequestException('im_group.not_found'); - } + $this->checkId($id); $groupCache = new ImGroupCache(); @@ -45,6 +34,8 @@ class ImGroup extends Validator public function checkGroup($id) { + $this->checkId($id); + $groupRepo = new ImGroupRepo(); $group = $groupRepo->findById($id); @@ -56,6 +47,19 @@ class ImGroup extends Validator return $group; } + public function checkId($id) + { + $id = intval($id); + + $maxGroupIdCache = new MaxImGroupIdCache(); + + $maxId = $maxGroupIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('im_group.not_found'); + } + } + public function checkName($name) { $value = $this->filter->sanitize($name, ['trim', 'string']); diff --git a/app/Validators/Package.php b/app/Validators/Package.php index 0a979e80..c6dcff5a 100644 --- a/app/Validators/Package.php +++ b/app/Validators/Package.php @@ -18,18 +18,7 @@ class Package extends Validator */ public function checkPackageCache($id) { - $id = intval($id); - - $maxPackageIdCache = new MaxPackageIdCache(); - - $maxPackageId = $maxPackageIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxPackageId) { - throw new BadRequestException('package.not_found'); - } + $this->checkId($id); $packageCache = new PackageCache(); @@ -44,6 +33,8 @@ class Package extends Validator public function checkPackage($id) { + $this->checkId($id); + $packageRepo = new PackageRepo(); $package = $packageRepo->findById($id); @@ -55,6 +46,19 @@ class Package extends Validator return $package; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxPackageIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('package.not_found'); + } + } + public function checkTitle($title) { $value = $this->filter->sanitize($title, ['trim', 'string']); diff --git a/app/Validators/Page.php b/app/Validators/Page.php index abc88d01..5925d029 100644 --- a/app/Validators/Page.php +++ b/app/Validators/Page.php @@ -18,18 +18,7 @@ class Page extends Validator */ public function checkPageCache($id) { - $id = intval($id); - - $maxPageIdCache = new MaxPageIdCache(); - - $maxPageId = $maxPageIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxPageId) { - throw new BadRequestException('page.not_found'); - } + $this->checkId($id); $pageCache = new PageCache(); @@ -44,6 +33,8 @@ class Page extends Validator public function checkPage($id) { + $this->checkId($id); + $pageRepo = new PageRepo(); $page = $pageRepo->findById($id); @@ -55,6 +46,19 @@ class Page extends Validator return $page; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxPageIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('page.not_found'); + } + } + public function checkTitle($title) { $value = $this->filter->sanitize($title, ['trim', 'string']); diff --git a/app/Validators/Topic.php b/app/Validators/Topic.php index e5231c6f..d31cf79a 100644 --- a/app/Validators/Topic.php +++ b/app/Validators/Topic.php @@ -18,18 +18,7 @@ class Topic extends Validator */ public function checkTopicCache($id) { - $id = intval($id); - - $maxTopicIdCache = new MaxTopicIdCache(); - - $maxTopicId = $maxTopicIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxTopicId) { - throw new BadRequestException('topic.not_found'); - } + $this->checkId($id); $topicCache = new TopicCache(); @@ -44,6 +33,8 @@ class Topic extends Validator public function checkTopic($id) { + $this->checkId($id); + $topicRepo = new TopicRepo(); $topic = $topicRepo->findById($id); @@ -55,6 +46,19 @@ class Topic extends Validator return $topic; } + public function checkId($id) + { + $id = intval($id); + + $maxIdCache = new MaxTopicIdCache(); + + $maxId = $maxIdCache->get(); + + if ($id < 1 || $id > $maxId) { + throw new BadRequestException('topic.not_found'); + } + } + public function checkTitle($title) { $value = $this->filter->sanitize($title, ['trim', 'string']); diff --git a/app/Validators/User.php b/app/Validators/User.php index 9b17500a..ac45c08b 100644 --- a/app/Validators/User.php +++ b/app/Validators/User.php @@ -21,18 +21,7 @@ class User extends Validator */ public function checkUserCache($id) { - $id = intval($id); - - $maxUserIdCache = new MaxUserIdCache(); - - $maxUserId = $maxUserIdCache->get(); - - /** - * 防止缓存穿透 - */ - if ($id < 1 || $id > $maxUserId) { - throw new BadRequestException('user.not_found'); - } + $this->checkId($id); $userCache = new UserCache(); @@ -47,6 +36,8 @@ class User extends Validator public function checkUser($id) { + $this->checkId($id); + $userRepo = new UserRepo(); $user = $userRepo->findById($id); @@ -58,6 +49,19 @@ class User extends Validator return $user; } + public function checkId($id) + { + $id = intval($id); + + $maxUserIdCache = new MaxUserIdCache(); + + $maxUserId = $maxUserIdCache->get(); + + if ($id < 1 || $id > $maxUserId) { + throw new BadRequestException('user.not_found'); + } + } + public function checkName($name) { $value = $this->filter->sanitize($name, ['trim', 'string']); diff --git a/public/static/home/css/common.css b/public/static/home/css/common.css index 0e860da7..967e023f 100644 --- a/public/static/home/css/common.css +++ b/public/static/home/css/common.css @@ -104,12 +104,6 @@ list-style: decimal; } -body { - display: flex; - flex-flow: column; - min-height: 100vh; -} - #header { left: 0; top: 0; @@ -122,7 +116,7 @@ body { #main { margin-top: 80px; margin-bottom: 30px; - flex: 1; + min-height: 560px; } #footer { @@ -1119,26 +1113,15 @@ body { text-align: center; } -.channel .alipay, .channel .wxpay { +.channel .btn-pay { width: 200px; height: 80px; display: inline-block; + line-height: 80px; margin-right: 15px; background-color: #f2f2f2; } -.channel .alipay { - background-image: url("/static/web/img/alipay.png"); - background-repeat: no-repeat; - background-position: center center; -} - -.channel .wxpay { - background-image: url("/static/web/img/wxpay.png"); - background-repeat: no-repeat; - background-position: center center; -} - .payment .footer { text-align: center; } diff --git a/public/static/home/js/order.pay.js b/public/static/home/js/order.pay.js index 535d7aec..3ec42261 100644 --- a/public/static/home/js/order.pay.js +++ b/public/static/home/js/order.pay.js @@ -16,7 +16,10 @@ layui.use(['jquery', 'layer'], function () { var qrHtml = $qrBlock.html(); if (qrHtml.length === 0) { - var postData = {order_sn: orderSn, channel: channel}; + var postData = { + order_sn: orderSn, + channel: channel === 'alipay' ? 1 : 2 + }; $.post(createUrl, postData, function (res) { qrHtml = '
    支付二维码
    '; showQrLayer(qrTitle, qrHtml);