Merge branch 'feature/question' into 'release/0718'
Feature/question See merge request !1478
Showing
3 changed files
with
21 additions
and
8 deletions
@@ -51,6 +51,10 @@ exports.check = (req, res, next) => { | @@ -51,6 +51,10 @@ exports.check = (req, res, next) => { | ||
51 | }; | 51 | }; |
52 | 52 | ||
53 | exports.geetestLoad = (req, res, next) => { | 53 | exports.geetestLoad = (req, res, next) => { |
54 | + if (req.user.uid) { | ||
55 | + return next(); | ||
56 | + } | ||
57 | + | ||
54 | req.yoho.captchaShow = false; | 58 | req.yoho.captchaShow = false; |
55 | res.locals.useGeetest = true; | 59 | res.locals.useGeetest = true; |
56 | 60 | ||
@@ -72,7 +76,7 @@ exports.geetestLoad = (req, res, next) => { | @@ -72,7 +76,7 @@ exports.geetestLoad = (req, res, next) => { | ||
72 | exports.geetestCheck = (req, res, next) => { | 76 | exports.geetestCheck = (req, res, next) => { |
73 | let testCode = req.body.yohobuy; | 77 | let testCode = req.body.yohobuy; |
74 | 78 | ||
75 | - if (testCode === global.yoho.config.testCode) { | 79 | + if (testCode === global.yoho.config.testCode || req.user.uid) { |
76 | return next(); | 80 | return next(); |
77 | } | 81 | } |
78 | 82 |
@@ -51,6 +51,7 @@ | @@ -51,6 +51,7 @@ | ||
51 | </div> | 51 | </div> |
52 | </div> | 52 | </div> |
53 | </div> | 53 | </div> |
54 | - | ||
55 | - <div data-userverify="{{captchaShow}}" data-geetest="{{useGeetest}}" id="js-img-check" {{#unless useGeetest}} class="full-img-verify" {{/unless}}></div> | 54 | + {{#if noLogin}} |
55 | + <div data-userverify="{{captchaShow}}" data-geetest="{{useGeetest}}" id="js-img-check" {{#unless useGeetest}} class="full-img-verify" {{/unless}}></div> | ||
56 | + {{/if}} | ||
56 | </div> | 57 | </div> |
@@ -6,18 +6,26 @@ let $ = require('yoho-jquery'), | @@ -6,18 +6,26 @@ let $ = require('yoho-jquery'), | ||
6 | share = require('common/share'), | 6 | share = require('common/share'), |
7 | Validate = require('plugin/validata'); | 7 | Validate = require('plugin/validata'); |
8 | 8 | ||
9 | -let validate = new Validate('#js-img-check', { | ||
10 | - useREM: { | ||
11 | - rootFontSize: 40, | ||
12 | - picWidth: 150 | 9 | +let validate = { |
10 | + getResults: () => { | ||
11 | + return Promise.resolve({}); | ||
13 | } | 12 | } |
14 | -}); | 13 | +}; |
15 | 14 | ||
16 | let question = { | 15 | let question = { |
17 | $base: $('#qs-wrap'), | 16 | $base: $('#qs-wrap'), |
18 | init: function() { | 17 | init: function() { |
19 | let that = this; | 18 | let that = this; |
20 | 19 | ||
20 | + if ($('#js-img-check').length) { | ||
21 | + validate = new Validate('#js-img-check', { | ||
22 | + useREM: { | ||
23 | + rootFontSize: 40, | ||
24 | + picWidth: 150 | ||
25 | + } | ||
26 | + }); | ||
27 | + } | ||
28 | + | ||
21 | this.$errTip = $('.error-tip'); | 29 | this.$errTip = $('.error-tip'); |
22 | this.$mobile = $('.user-mobile > input'); | 30 | this.$mobile = $('.user-mobile > input'); |
23 | this.$item = $('.qs-item, .sub-qs-item', this.$base); | 31 | this.$item = $('.qs-item, .sub-qs-item', this.$base); |
-
Please register or login to post a comment