qr-bills 0.2.2 → 0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 673b6c6cd7948ecf8121a1cff9462c6d60a7c8b1022078fae2eb463919885dc7
-  data.tar.gz: 765e8d32630c70b9b8441e0cc8613821ba8e683bd9324293adc3ebdea4ec785d
+  metadata.gz: f0f5417c130ea2bb955adc879dabbc2e93f470c3990bd7b79cc5c9d069a92aca
+  data.tar.gz: 412fd936ffa901e2d83909215c5dfe5e700945d76ae0ca8b3b4e4ea3d48f8740
 SHA512:
-  metadata.gz: bcb1a9f90fae5d55d14d2452212a47a29fa6049e9f1f86cc9ed682824424614a5909095b69aa0adc4f2b8d1f092fcc9fd56316b8742c559ba571acaaaba7a282
-  data.tar.gz: 58723203eeb46a9948ed5d37fcb35603c2e6abb7a294d351c30c7f3e26629fc0a4eecfd493aa947cef794a25fdc56c6a73432f57be3a373ce5c9fe2146c1b28d
+  metadata.gz: a1448d54d8fddc9ae2e11fa8d900fa21f0a2c0d501360a6386636a9650200c53f3b487c0aa05d1acfb8909e2ab816ee57c1c70ce88d08f84ecfe903c1f09883e
+  data.tar.gz: 5787cc90e6a2546736376f7268bc13e044cfd06b0783b4876e77d12ff1d8f465c02cc804377e6cfe64074ac329e3f5025837f5de221d3f2391f6d169d8820209

data/lib/qr-bills.rb

@@ -4,41 +4,32 @@ require 'qr-bills/qr-params'
 require 'qr-bills/qr-html-layout'
 require 'qr-bills/qr-creditor-reference'
 
-class QRBills
+module QRBills
   def self.generate(qr_params)
+    raise ArgumentError, "#{QRExceptions::INVALID_PARAMETERS}: bill type param not set" unless qr_params.has_key?(:bill_type)
+    raise ArgumentError, "#{QRExceptions::INVALID_PARAMETERS}: validation failed" unless QRParams.valid?(qr_params)
 
-    # params validation
-    if qr_params.has_key?(:bill_type)
-
-      if !QRParams.valid?(qr_params)
-        raise QRExceptions::INVALID_PARAMETERS + ": params validation check failed"
-      end
+    # init translator sets
+    %i[it en de fr].each do |locale|
+      locale_file = File.join(qr_params[:locales][:path], "qrbills.#{locale}.yml")
 
-      if !qr_params[:output_params][:format] == "html"
-        raise QRExceptions::NOT_SUPPORTED + ": html is the only output format supported so far"
-      end
+      I18n.load_path << locale_file
+    end
 
+    output = case qr_params[:output_params][:format]
+    when 'html'
+      QRHTMLLayout.create(qr_params)
+    when 'qrcode_png'
+      QRGenerator.create(qr_params, qr_params[:qrcode_filepath])
     else
-      raise QRExceptions::INVALID_PARAMETERS + ": bill type param not set"
+      raise ArgumentError, "#{QRExceptions::NOT_SUPPORTED}: #{qr_params[:output_params][:format]} is not yet supported"
     end
 
-    # init translator sets
-    I18n.load_path << File.join(qr_params[:locales][:path], "qrbills.it.yml")
-    I18n.load_path << File.join(qr_params[:locales][:path], "qrbills.en.yml")
-    I18n.load_path << File.join(qr_params[:locales][:path], "qrbills.de.yml")
-    I18n.load_path << File.join(qr_params[:locales][:path], "qrbills.fr.yml")
-    I18n.default_locale = :it
-
-    bill = { 
-      params: qr_params,
-      output: QRHTMLLayout.create(qr_params) 
-    }
-
-    return bill
+    { params: qr_params, output: output }
   end
 
   def self.create_creditor_reference(reference)
-    return QRCreditorReference.create(reference)
+    QRCreditorReference.create(reference)
   end
 
   def self.get_qr_params

data/lib/qr-bills/qr-creditor-reference.rb

@@ -1,7 +1,7 @@
 require 'qr-bills/qr-exceptions'
 
 # implement Creditor Reference ISO 11649 generator
-class QRCreditorReference
+module QRCreditorReference
   PREFIX = "RF"
 
   @char_values = {

data/lib/qr-bills/qr-exceptions.rb

@@ -1,4 +1,4 @@
-class QRExceptions
+module QRExceptions
   EXCEPTION_PREFIX = "QR-bill"
 
   INVALID_PARAMETERS = EXCEPTION_PREFIX + " invalid parameters"

data/lib/qr-bills/qr-generator.rb

@@ -1,8 +1,6 @@
 require 'rqrcode'
-require 'rmagick'
-include Magick
 
-class QRGenerator
+module QRGenerator
   # payload:
   #  "SPC\r\n" +    # indicator for swiss qr code: SPC (swiss payments code)
   #  "0200\r\n" +   # version of the specifications, 0200 = v2.0
@@ -39,11 +37,6 @@ class QRGenerator
   #  "eBill/B/41010560425610173";                 # alternative scheme paramaters, max 100 chars
  
   def self.create(params, qrcode_path)
-    create_qr(params, qrcode_path)
-    add_swiss_cross(qrcode_path, qrcode_path)
-  end
-
-  def self.create_qr(params, qrcode_path)
     payload = "SPC\r\n"
     payload += "0200\r\n"
     payload += "1\r\n"
@@ -93,13 +86,8 @@ class QRGenerator
       size: 1024,
     )
 
-    IO.binwrite(qrcode_path, png.to_s)
-  end
-
-  def self.add_swiss_cross(qrcode_path, final_path)
-    qr_image    = Image.read(qrcode_path)[0]
-    swiss_cross = Image.read(File.expand_path("#{File.dirname(__FILE__)}/../../web/assets/images/swiss_cross.png"))[0]
-    final_qr    = qr_image.composite(swiss_cross, CenterGravity, OverCompositeOp)
-    final_qr.write(final_path)
+    swiss_cross = ChunkyPNG::Image.from_file(File.expand_path("#{File.dirname(__FILE__)}/../../web/assets/images/swiss_cross.png"))
+    final_qr    = png.compose!(swiss_cross, (png.width - swiss_cross.width) / 2,  (png.height - swiss_cross.height) / 2)
+    final_qr.save(qrcode_path)
   end
 end

data/lib/qr-bills/qr-html-layout.rb

@@ -1,6 +1,6 @@
 require 'qr-bills/qr-generator'
 
-class QRHTMLLayout
+module QRHTMLLayout
 
   def self.create(params)
     QRGenerator.create(params, params[:qrcode_filepath])

data/lib/qr-bills/qr-params.rb

@@ -1,7 +1,7 @@
-class QRParams
+module QRParams
   QR_BILL_WITH_QR_REFERENCE       = "orange_with_reference"
   QR_BILL_WITH_CREDITOR_REFERENCE = "red_with_reference"
-  QR_BILL_WITOUTH_REFERENCE       = "red_without_reference"
+  QR_BILL_WITHOUT_REFERENCE       = "red_without_reference"
 
   def self.get_qr_params
     {
@@ -56,43 +56,38 @@ class QRParams
   end
 
   def self.valid?(params)
-    if params.has_key?(:bill_type)
+    return false unless params.key?(:bill_type)
+    return false unless QRParams.base_params_valid?(params)
 
-      if !QRParams.base_params_valid?(params)
-        return false
-      end
-
-      if params[:bill_type] == QR_BILL_WITH_QR_REFERENCE
-        return QRParams.qr_bill_with_qr_reference_valid?(params)
-      elsif params[:bill_type] == QR_BILL_WITH_CREDITOR_REFERENCE
-        return QRParams.qr_bill_with_creditor_reference_valid?(params)
-      elsif params[:bill_type] == QR_BILL_WITOUTH_REFERENCE
-        return QRParams.qr_bill_without_reference_valid?(params)
-      else
-        return false
-      end
+    case params[:bill_type]
+    when QRParams::QR_BILL_WITH_QR_REFERENCE
+      QRParams.qr_bill_with_qr_reference_valid?(params)
+    when QRParams::QR_BILL_WITH_CREDITOR_REFERENCE
+      QRParams.qr_bill_with_creditor_reference_valid?(params)
+    when QRParams::QR_BILL_WITHOUT_REFERENCE
+      QRParams.qr_bill_without_reference_valid?(params)
     else
-      return false
+      false
     end
   end
-
+  
   def self.base_params_valid?(params)
-    # todo
-    return true
+    # TODO
+    true
   end
 
   def self.qr_bill_with_qr_reference_valid?(params)
-    # todo
-    return true
+    # TODO
+    true
   end
 
   def self.qr_bill_with_creditor_reference_valid?(params)
-    # todo
-    return true
+    # TODO
+    true
   end
 
   def self.qr_bill_without_reference_valid?(params)
-    # todo
-    return true
+    # TODO
+    true
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: qr-bills
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: '0.3'
 platform: ruby
 authors:
 - Damiano Radice
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-07-11 00:00:00.000000000 Z
+date: 2021-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: i18n
@@ -50,26 +50,6 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '2'
-- !ruby/object:Gem::Dependency
-  name: rmagick
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 4.1.2
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 4.1.2
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -100,7 +80,7 @@ dependencies:
         version: '13.0'
 description: 'QR-bills support for swiss payments, for full documentation please refer
   to github repo: https://github.com/damoiser/qr-bills'
-email: damoiser+qrbill@gmail.com
+email: dam.radice@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -133,7 +113,7 @@ metadata:
   homepage_uri: https://github.com/damoiser/qr-bills
   source_code_uri: https://github.com/damoiser/qr-bills
   wiki_uri: https://github.com/damoiser/qr-bills
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -149,7 +129,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.1.2
-signing_key: 
+signing_key:
 specification_version: 4
 summary: QR-bills support for swiss payments
 test_files: []