lib/onebox/engine/stack_exchange_onebox.rb in onebox-1.5.43 vs lib/onebox/engine/stack_exchange_onebox.rb in onebox-1.5.44

- old
+ new

@@ -4,14 +4,15 @@ include Engine include LayoutSupport include JSON def self.domains - %w(stackexchange stackoverflow superuser serverfault askubuntu) + %w(stackexchange.com stackoverflow.com superuser.com serverfault.com askubuntu.com stackapps.com mathoverflow.net) + .map { |domain| Regexp.escape(domain) } end - matches_regexp /^https?:\/\/(?:(?:(?<subsubdomain>\w*)\.)?(?<subdomain>\w*)\.)?(?<domain>#{domains.join('|')})\.com\/(?:questions|q)\/(?<question>\d*)/ + matches_regexp /^https?:\/\/(?:(?:(?<subsubdomain>\w*)\.)?(?<subdomain>\w*)\.)?(?<domain>#{domains.join('|')})\/((?:questions|q)\/(?<question_id>\d*)(\/.*\/(?<answer_id1>\d*))?|(a\/(?<answer_id2>\d*)))/ def always_https? uri.host.split('.').length <= 3 end @@ -21,21 +22,30 @@ @match ||= @url.match(@@matcher) end def url domain = URI(@url).host - "https://api.stackexchange.com/2.1/questions/#{match[:question]}?site=#{domain}" + question_id = match[:question_id] + answer_id = match[:answer_id2] || match[:answer_id1] + + if answer_id + "https://api.stackexchange.com/2.2/answers/#{answer_id}?site=#{domain}&filter=!.FjueITQdx6-Rq3Ue9PWG.QZ2WNdW" + else + "https://api.stackexchange.com/2.2/questions/#{question_id}?site=#{domain}&filter=!5-duuxrJa-iw9oVvOA(JNimB5VIisYwZgwcfNI" + end end def data return @data if @data result = raw['items'][0] if result result['creation_date'] = - Time.at(result['creation_date'].to_i).strftime("%I:%M%p - %d %b %y") + Time.at(result['creation_date'].to_i).strftime("%I:%M%p - %d %b %y") result['tags'] = result['tags'].take(4).join(', ') + result['is_answer'] = result.key?('answer_id') + result['is_question'] = result.key?('question_id') end @data = result end end end