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