snoo 0.1.0.pre.1 → 0.1.0.pre.2

data/lib/snoo/flair.rb

@@ -12,8 +12,6 @@ module Snoo
     # @return (see #clear_sessions)
     def clear_flair_templates type, subreddit
       logged_in?
-      tests = ['USER_FLAIR', 'LINK_FLAIR']
-      raise ArgumentError, 'type needs to be either USER_FLAIR or LINK_FLAIR' unless tests.include?(type)
       post('/api/clearflairtemplates', body: { flair_type: type, r: subreddit, uh: @modhash})
     end
 
@@ -47,7 +45,6 @@ module Snoo
     # @return (see #clear_sessions)
     def flair subreddit, opts = {}
       logged_in?
-      raise ArgumentError, "link or name, not both" if opts[:link] && opts[:name]
       params = {
         r: subreddit,
         uh: @modhash,
@@ -103,7 +100,6 @@ module Snoo
     # @return (see #clear_sessions)
     def get_flair_list subreddit, opts = {}
       logged_in?
-      raise ArgumentError, 'limit is too high/low' unless (1..1000).include?(opts[:limit]) or opts[:limit].nil?
       query = {
         limit: 1000,
         uh: @modhash
@@ -131,10 +127,6 @@ module Snoo
         r: subreddit
       }
       params.merge! opts
-      test = ['USER_FLAIR', 'LINK_FLAIR']
-      raise ArgumentError, 'type is either USER_FLAIR or LINK_FLAIR' unless test.include?(params[:flair_type])
-      test = [true, false]
-      raise ArgumentError, 'user_editable needs to be true or false' unless test.include?(params[:text_editable])
 
       post('/api/flairtemplate', body: params)
     end
@@ -150,7 +142,6 @@ module Snoo
     # @return (see #clear_sessions)
     def select_flair_template template_id, subreddit, opts = {}
       logged_in?
-      raise ArgumentError, 'link or user, not both' if opts[:link] && opts[:user]
       params = {
         flair_template_id: template_id,
         uh: @modhash,
@@ -167,8 +158,6 @@ module Snoo
     # @return (see #clear_sessions)
     def flair_toggle enabled, subreddit
       logged_in?
-      test = [true, false]
-      raise ArgumentError, 'enabled must be boolean' unless test.include?(enabled)
       post('/api/setflairenabled', body: {flair_enabled: enabled, uh: @modhash, r: subreddit})
     end
 

data/lib/snoo/links_comments.rb

@@ -52,7 +52,6 @@ module Snoo
     # @option opts [Fixnum] :limit The number of things to return. Go too high and the API will ignore you
     # @return (see #clear_sessions)
     def info opts = {}
-      raise ArgumentError, 'url or id, not both' if opts[:id] && opts[:url]
       query = { limit: 100 }
       query.merge! opts
       get('/api/info.json', query: query)
@@ -95,7 +94,6 @@ module Snoo
     # @return (see #clear_sessions)
     def submit title, subreddit, opts = {}
       logged_in?
-      raise ArgumentError, 'url or text, not both' if opts[:url] && opts[:text]
       post = {
         title: title,
         sr: subreddit,
@@ -132,7 +130,6 @@ module Snoo
     # @return (see #clear_sessions)
     def vote direction, id
       logged_in?
-      raise ArgumentError, "direction needs to be one of -1, 0, or 1 (was #{direction}" unless (-1..1).include?(direction)
       post('/api/vote', body: {id: id, dir: direction, uh: @modhash})
     end
 

data/lib/snoo/listings.rb

@@ -17,8 +17,6 @@ module Snoo
     # @option opts [old, new, hot, top, controversial, best] :sort The sort used.
     # @return (see #clear_sessions)
     def get_comments opts = {}
-      sorts = %w{old new hot top controversial best}
-      raise ArgumentError, "sort cannot be #{sort}" unless sorts.include?(opts[:sort]) or opts[:sort].nil?
       query = { limit: 100 }
       query.merge! opts
       url = "%s/comments/%s%s.json" % [('/r/' + opts[:subreddit] if opts[:subreddit]), opts[:link_id], ('/' + opts[:comment_id] if opts[:comment_id])]
@@ -37,20 +35,6 @@ module Snoo
     # @option opts [String] :before Get things *before* this thing id
     # @return (see #clear_sessions)
     def get_listing opts = {}
-      pages = %w{new controversial top saved}
-      sorts = %w{new rising}
-      times = %w{hour day week month year}
-      # Invalid Page
-      raise ArgumentError, "page must be #{pages * ', '}, is #{opts[:page]}" unless pages.include?(opts[:page]) or opts[:page].nil?
-      # Invalid Sort
-      raise ArgumentError, "sort must be one of #{sorts * ', '}, is #{opts[:sort]}" unless sorts.include?(opts[:sort]) or opts[:sort].nil?
-      # Sort on useless page
-      raise ArgumentError, "sort can only be used on page = 'new'" if opts[:page] != 'new' && opts[:sort]
-      # Invalid time
-      raise ArgumentError, "time can only be one of #{times * ', '}, is #{opts[:time]}" unless times.include?(opts[:time]) or opts[:time].nil?
-      # Invalid limit
-      raise ArgumentError, "limit cannot be outside 1..100, is #{opts[:limit]}" unless (1..100).include?(opts[:limit]) or opts[:limit].nil?
-
       # Build the basic url
       url = "%s/%s.json" % [('/r/' + opts[:subreddit] if opts[:subreddit] ), (opts[:page] if opts[:page])]
       # Delete subreddit and page from the hash, they dont belong in the query

data/lib/snoo/moderation.rb

@@ -21,7 +21,6 @@ module Snoo
     def distinguish id, how = "yes"
       logged_in?
       hows = %w{yes no admin special}
-      raise ArgumentError, "how should be one of #{hows * ', '}, is #{how}" unless hows.include?(how)
       post('/api/distinguish', body: {id: id, how: how, uh: @modhash})
     end
 
@@ -52,8 +51,6 @@ module Snoo
     # @return (see #clear_sessions)
     def remove id, spam = false
       logged_in?
-      spams = [true, false]
-      raise ArgumentError, "spam should be boolean, is #{spam}" unless spams.include?(spam)
       post('/api/remove', body: {id: id, spam: spam, uh: @modhash})
     end
 

data/lib/snoo/subreddits.rb

@@ -23,6 +23,14 @@ module Snoo
       post('/api/delete_sr_image', body: {r: subreddit, img_name: image_name, uh: @modhash})
     end
 
+    # Gets a hash of the subreddit settings
+    # Returns a webserver error (404) if you don't have moderator permissions on said subreddit
+    #
+    # @param subreddit [String] the subreddit to fetch data from
+    def get_subreddit_settings subreddit
+      logged_in?
+      get("/r/#{subreddit}/about/edit/.json")
+    end
     # @todo test if every param is actually required
     # Sets subreddit settings.
     #
@@ -41,12 +49,6 @@ module Snoo
     # @return (see #clear_sessions)
     def subreddit_settings subreddit, opts = {}
       logged_in?
-      bool = [true, false]
-      raise ArgumentError, "type must be one of public, private, restricted, is #{opts[:type]}" unless %w{public private restricted}.include?(opts[:type]) or opts[:type].nil?
-      raise ArgumentError, "post_type must be one of any, link, self; is #{opts[:link_type]}" unless %w{any link self}.include?(opts[:link_type]) or opts[:link_type].nil?
-      raise ArgumentError, "allow_frontpage must be boolean" unless bool.include?(opts[:allow_top]) or opts[:allow_top].nil?
-      raise ArgumentError, "show_media must be boolean" unless bool.include?(opts[:show_media]) or opts[:show_media].nil?
-      raise ArgumentError, "adult must be boolean" unless bool.include?(opts[:over_18]) or opts[:over_18].nil?
       params = {
         type: 'public',
         link_type: 'any',
@@ -68,7 +70,7 @@ module Snoo
     # @return (see #clear_sessions)
     def set_stylesheet stylesheet, subreddit
       logged_in?
-      post('/api/subreddit_stylesheet', body: {op: "save", stylesheet_contents: stylesheet, uh: @modhash})
+      post('/api/subreddit_stylesheet', body: {op: 'save', r: subreddit, stylesheet_contents: stylesheet, uh: @modhash})
     end
 
     # Subscribe to a subreddit
@@ -78,7 +80,6 @@ module Snoo
     # @return (see #clear_sessions)
     def subscribe subreddit, action = "sub"
       logged_in?
-      raise ArgumentError, "action must be one of sub, unsub; is #{action}" unless %w{sub unsub}.include?(action)
       post('/api/subscribe', body: {action: action, sr: subreddit, uh: @modhash})
     end
 
@@ -119,8 +120,6 @@ module Snoo
     # @return (see #clear_sessions)
     def my_reddits opts = {}
       logged_in?
-      raise ArgumentError, "condition must be one of subscriber, contributor, moderator; is #{opts[:condition]}" unless %w{subscriber contributor moderator}.include?(opts[:condition]) or opts[:condition].nil?
-      raise ArgumentError, "limit must be within 1..100; is #{opts[:limit]}" unless (1..100).include?(opts[:limit]) or opts[:limit].nil?
       url = "/reddits/mine/%s.json" % (opts[:condition] if opts[:condition])
       opts.delete :condition
       query = opts
@@ -136,9 +135,6 @@ module Snoo
     # @option opts [String] :before Return subreddits *before* this id.
     # @return (see #clear_sessions)
     def get_reddits opts = {}
-      raise ArgumentError, "condition must be one of popular, new, banned; is #{opts[:condition]}" unless %w{popular new banned}.include?(opts[:condition]) or opts[:condition].nil?
-      raise ArgumentError, "limit must be within 1..100; is #{opts[:limit]}" unless (1..100).include?(opts[:limit]) or opts[:limit].nil?
-
       url = "/reddits/%s.json" % (opts[:condition] if opts[:condition])
       opts.delete :condition
       query = opts
@@ -155,8 +151,6 @@ module Snoo
     # @option opts [String] :before Return subreddits *before* this id.
     # @return (see #clear_sessions)
     def search_reddits q, opts = {}
-      raise ArgumentError, "limit must be within 1..100; is #{opts[:limit]}" unless (1..100).include?(opts[:limit]) or opts[:limit].nil?
-
       query = {q: q}
       query.merge! opts
       get('/reddits/search.json', query: query)

data/lib/snoo/version.rb

@@ -1,4 +1,4 @@
 module Snoo
   # The version string (duh)
-  VERSION = "0.1.0.pre.1"
+  VERSION = "0.1.0.pre.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: snoo
 version: !ruby/object:Gem::Version
-  version: 0.1.0.pre.1
+  version: 0.1.0.pre.2
   prerelease: 6
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-17 00:00:00.000000000 Z
+date: 2012-10-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty