lib/lolcommits/plugin/slack.rb in lolcommits-slack-0.0.2 vs lib/lolcommits/plugin/slack.rb in lolcommits-slack-0.0.3

@@ -9,21 +9,10 @@
 
       # Number of times to retry if RestClient.post fails
       RETRY_COUNT = 2
 
       ##
-      # Returns the name of the plugin.
-      #
-      # Identifies the plugin to lolcommits.
-      #
-      # @return [String] the plugin name
-      #
-      def self.name
-        'slack'
-      end
-
-      ##
       # Returns position(s) of when this plugin should run during the capture
       # process.
       #
       # The lolcommit is posted to Slack when all proccessing has completed and
       # the capture is ready.
@@ -50,15 +39,15 @@
         begin
           print "Posting to Slack ... "
           response = RestClient.post(
             ENDPOINT_URL,
             file: File.new(runner.main_image),
-            token: configuration['access_token'],
+            token: configuration[:access_token],
             filetype: 'jpg',
             filename: runner.sha,
             title: runner.message + "[#{runner.vcs_info.repo}]",
-            channels: configuration['channels']
+            channels: configuration[:channels]
           )
 
           debug response
           print "done!\n"
         rescue => e
@@ -84,37 +73,26 @@
       # @return [Hash] a hash of configured plugin options
       #
       def configure_options!
         options = super
 
-        if options['enabled']
+        if options[:enabled]
           print "open the url below and issue a token for your user:\n"
           print "https://api.slack.com/custom-integrations/legacy-tokens\n"
           print "enter the generated token below, then press enter: (e.g. xxxx-xxxxxxxxx-xxxx) \n"
           code = parse_user_input(gets.strip)
 
           print "enter a comma-seperated list of channel ids to post images in, then press enter: (e.g. c1234567890,c1234567890)\n"
           print "note: you must use channel ids (not channel names). grab them from here; https://api.slack.com/methods/channels.list/test\n"
           channels = parse_user_input(gets.strip)
 
           options.merge!(
-            'access_token' => code,
-            'channels'     => channels
+            access_token: code,
+            channels: channels
           )
         end
 
         options
-      end
-
-      ##
-      # Returns true/false indicating if the plugin has been configured.
-      #
-      # Checks the `access_token` ond `channels` options have been set.
-      #
-      # @return [Boolean] true/false
-      #
-      def configured?
-        !configuration['access_token'].nil? && !configuration['channels'].nil?
       end
     end
   end
 end