ad_localize 4.0.1 → 4.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4912c63504f0bb4c170b0d58a2e4fdc2df11a66b69ff300644c098e262ac36bf
-  data.tar.gz: 8ba77200d900a9c379849b876a733fd201eec256ce473365b19909287671feb1
+  metadata.gz: 5dc831e39e21e628ccb906bb9c1b6c3d413e6c182384b6e679a7586e0826d93a
+  data.tar.gz: 0e3a03ab471b2a45e1b23b44a67d196bf609503ff51364b81fe886b03fc895ba
 SHA512:
-  metadata.gz: 82408859e066f57b962fe07df4e577efa92ae1621b8b6f6a5419316266fd811ccc8e3c832db98ded07e425642cd192de6ed579efb13310173da217892601ebca
-  data.tar.gz: 534c388c96c9cc11b4dee70e6f4537ec095ee9218d5d3d6beaa23825ec8ac570810eae6abea32870a00be332638e0856b8ba85c9d43187fd790af24b6bd34db4
+  metadata.gz: 571e52661789dd15c8a24958a2a2cae26a166e4081b7739ee09fb5df6993ec9658c26e3df3aebcdf458d9980ebb4e2a32bb1eefca294daa2c3abe1396d1faf6f
+  data.tar.gz: e66e4710b854bf15e161e6149f2f2d20e9ff54cc0ddcc0207d2bc421dd935a35dbd419069b37e9b9fd3f3549222637a0262f24c2f486e83673589eaf9f3fdae1

data/CHANGELOG.md

@@ -4,6 +4,31 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [4.0.5] - 2020-01-25
+### Fixed
+  - Fix google spreadsheet sheet id comparison with sheet_ids option value
+
+## [4.0.5] - 2020-11-12
+### Fixed
+  - Fix csv file type check (also add compatibility with macOS Big Sur)
+  
+## [4.0.4] - 2020-11-09
+### Fixed
+  - Fix android special character escaping [#56](https://github.com/applidium/ad_localize/issues/56)
+  - Fix platform filter on export [#55](https://github.com/applidium/ad_localize/issues/55)
+  
+### Changed
+  - Use default terminal color for debug log
+
+## [4.0.3] - 2020-10-27
+### Fixed
+  - Fix CSV detection and remove dependency to MimeType gem
+  - Fix drive key detection in options
+
+## [4.0.2] - 2020-10-26
+### Fixed
+  - Use percent HTML char for escaping '%' on android. [#49](https://github.com/applidium/ad_localize/pull/49) by [flolom](https://github.com/flolom)
+
 ## [4.0.1] - 2020-10-19
 ### Fixed
   - delete downloaded files even when any of them is a CSV file
@@ -13,7 +38,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Precedence to csv files. Only CSV files will be exported if both csv file and google spreadsheet are provided
   - In case of multiple CSVs or sheet ids, you should be aware that all sources will be merged. By default the merge policy to keep the first wording translation for each key
   - New architecture. Separate export process responsibilities in dedicated classes. Fixes [#48](https://github.com/applidium/ad_localize/issues/48), [#20](https://github.com/applidium/ad_localize/issues/20)
- 
+
 ### Changed
   - it is now possible to provide an output path path that does not exist
   - if a service account configuration is provided, there won't be any files downloaded
@@ -24,8 +49,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - tests for ad_localize class are restricted to the minimum
   - No more file generated when there are no data to export
   - Less verbose logs. Now, they only describe the different steps of the export process
-  - do not add `InfoPlist` translations to android `strings.xml` 
-  
+  - do not add `InfoPlist` translations to android `strings.xml`
+
 ### Added
   - -e, -export_all_sheets option export all sheets from a spreadsheet by [@sjcqs](https://github.com/sjcqs)
   - -m, --merge-option to select the merge policy (`keep` or `replace`) by [@sjcqs](https://github.com/sjcqs)
@@ -37,7 +62,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - dedicated folder for fixture files
   - add ability to use AdLocalize in a Ruby program
   - add documentation for JSON and YAML support. Fixes [#23](https://github.com/applidium/ad_localize/issues/23)
-  
+
 ### Fixed
   - comments will now be added to iOS and Android plural files
   - do not remove existing files in export output folder. Fixes [#40](https://github.com/applidium/ad_localize/issues/40)
@@ -46,8 +71,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - no more option -a option to indicate that a service account configuration will be provided. If set, the environment variable `GCLOUD_CLIENT_SECRET` content will be used
   - no more substitution of empty wording with by "Missing Translation" when using the option -d
   - no more Makefile, the Rakefile is sufficient. Use `bundle exec rake -T` to display the available commands
-  - no more check for ordered interpolation variables in translations 
-  
+  - no more check for ordered interpolation variables in translations
+
 ## [3.6.0] - 2020-06-23
 ### Added
   - add documentation for service account usage by [@sjcqs](https://github.com/sjcqs)

data/Gemfile.lock

@@ -1,12 +1,11 @@
 PATH
   remote: .
   specs:
-    ad_localize (4.0.1)
+    ad_localize (4.0.6)
       activesupport (>= 5.2, < 7.0)
       colorize (~> 0.8)
       google-api-client (~> 0.34)
       googleauth (~> 0.12)
-      mime-types (~> 3.3)
       nokogiri (~> 1.10)
 
 GEM
@@ -28,17 +27,19 @@ GEM
     declarative (0.0.20)
     declarative-option (0.1.0)
     diffy (3.3.0)
-    faraday (1.0.1)
+    faraday (1.1.0)
       multipart-post (>= 1.2, < 3)
-    google-api-client (0.44.1)
+      ruby2_keywords
+    google-api-client (0.46.2)
       addressable (~> 2.5, >= 2.5.1)
       googleauth (~> 0.9)
       httpclient (>= 2.8.1, < 3.0)
       mini_mime (~> 1.0)
       representable (~> 3.0)
       retriable (>= 2.0, < 4.0)
+      rexml
       signet (~> 0.12)
-    googleauth (0.13.1)
+    googleauth (0.14.0)
       faraday (>= 0.17.3, < 2.0)
       jwt (>= 1.4, < 3.0)
       memoist (~> 0.16)
@@ -50,11 +51,8 @@ GEM
       concurrent-ruby (~> 1.0)
     jwt (2.2.2)
     memoist (0.16.2)
-    mime-types (3.3.1)
-      mime-types-data (~> 3.2015)
-    mime-types-data (3.2020.0512)
     mini_mime (1.0.2)
-    mini_portile2 (2.4.0)
+    mini_portile2 (2.5.0)
     minitest (5.14.0)
     minitest-reporters (1.4.2)
       ansi
@@ -63,17 +61,21 @@ GEM
       ruby-progressbar
     multi_json (1.15.0)
     multipart-post (2.1.1)
-    nokogiri (1.10.10)
-      mini_portile2 (~> 2.4.0)
+    nokogiri (1.11.1)
+      mini_portile2 (~> 2.5.0)
+      racc (~> 1.4)
     os (1.1.1)
     public_suffix (4.0.6)
+    racc (1.5.2)
     rake (12.3.3)
     representable (3.0.4)
       declarative (< 0.1.0)
       declarative-option (< 0.2.0)
       uber (< 0.2.0)
     retriable (3.1.2)
+    rexml (3.2.4)
     ruby-progressbar (1.10.1)
+    ruby2_keywords (0.0.2)
     signet (0.14.0)
       addressable (~> 2.3)
       faraday (>= 0.17.3, < 2.0)

data/README.md

@@ -31,7 +31,7 @@ $ gem install ad_localize
 $ ad_localize -h
 ```
 
-* Export wording from a google spreadsheet, default tab. The spreadsheet key and sheet_id are available in the spreadsheet url. For example `https://docs.google.com/spreadsheets/d/<your-spreadsheet-drive-key>/edit#gid=<sheet-id>`.
+* Export wording from a public google spreadsheet, default tab. The spreadsheet key and sheet_id are available in the spreadsheet url. For example `https://docs.google.com/spreadsheets/d/<your-spreadsheet-drive-key>/edit#gid=<sheet-id>`.
 ```
 $ ad_localize -k <your-spreadsheet-drive-key>
 ```
@@ -41,6 +41,11 @@ $ ad_localize -k <your-spreadsheet-drive-key>
 $ ad_localize -k <your-spreadsheet-drive-key> -s <comma-separated-sheet-id-list>
 ```
 
+* Export wording from a private google spreadsheet. It requires a [Google Cloud Service Account](#using-a-google-cloud-service-account). 
+```
+$ GCLOUD_CLIENT_SECRET=$(cat <path-to-client-secret.json>) ad_localize -k <your-spreadsheet-drive-key>
+```
+
 * Export wording from all sheets in a google spreadsheet. It requires a [Google Cloud Service Account](#using-a-google-cloud-service-account). 
 ```
 $ GCLOUD_CLIENT_SECRET=$(cat <path-to-client-secret.json>) ad_localize -k <your-spreadsheet-drive-key> -e

data/ad_localize.gemspec

@@ -51,7 +51,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'colorize', '~> 0.8'
   spec.add_dependency 'google-api-client', '~> 0.34'
   spec.add_dependency 'googleauth', '~> 0.12'
-  spec.add_dependency 'mime-types', '~> 3.3'
 
   spec.required_ruby_version = '~> 2.3'
 end

data/lib/ad_localize.rb

@@ -9,7 +9,6 @@ require 'logger'
 require 'colorize'
 require 'optparse'
 require 'nokogiri'
-require 'mime/types'
 require 'erb'
 require 'googleauth'
 require 'google/apis/sheets_v4'

data/lib/ad_localize/ad_logger.rb

@@ -18,7 +18,7 @@ module AdLocalize
     end
 
     def debug(text)
-      log(level: Logger::DEBUG, text: text.black)
+      log(level: Logger::DEBUG, text: text)
     end
 
     def info!

data/lib/ad_localize/mappers/android_translation_mapper.rb

@@ -5,12 +5,16 @@ module AdLocalize
 
       def sanitize_value(value:)
         return if value.blank?
-        processedValue = value.gsub(/(?<!\\)'/, "\\\\'") # match ' unless there is a \ before
-        processedValue = processedValue.gsub(/(?<!\\)\"/, "\\\"") # match " unless there is a \ before
+        processedValue = value.gsub(/(?<!\\)'/, '&#39;') # match ' unless there is a \ before
+        processedValue = processedValue.gsub(/(?<!\\)\"/, '&#34;') # match " unless there is a \ before
+        processedValue = processedValue.gsub(">", '&gt;')
+        processedValue = processedValue.gsub("<", '&lt;')
         processedValue = processedValue.gsub(/(%(\d+\$)?@)/, '%\2s') # should match values like %1$s and %s
         processedValue = processedValue.gsub(/(%((\d+\$)?(\d+)?)i)/, '%\2d') # should match values like %i, %3$i, %01i, %1$02i
-        processedValue = processedValue.gsub(/%(?!((\d+\$)?(s|(\d+)?d)))/, '%%') # negative lookahead: identifies when user really wants to display a %
+        processedValue = processedValue.gsub(/%(?!((\d+\$)?(s|(\d+)?d)))/, '&#37;') # negative lookahead: identifies when user really wants to display a %
         processedValue = processedValue.gsub("\\U", "\\u")
+        processedValue = processedValue.gsub(/&(?!((#\d+)|(\w+));)/, '&#38;')
+        processedValue = processedValue.gsub(/&/) { |match| match.replace('\&') }
         "\"#{processedValue}\""
       end
     end

data/lib/ad_localize/mappers/options_to_export_request.rb

@@ -3,7 +3,7 @@ module AdLocalize
     class OptionsToExportRequest
       def map(options:)
         Requests::ExportRequest.new(
-          platform_codes: options[:only],
+          platforms: options[:only],
           g_spreadsheet_options: map_g_spreadsheet_options(options: options),
           verbose: options[:debug],
           output_path: options[:'target-dir'],
@@ -15,7 +15,7 @@ module AdLocalize
       private
 
       def map_g_spreadsheet_options(options:)
-        return unless options[:drive_key]
+        return unless options[:'drive-key']
         Requests::GSpreadsheetOptions.new(
           spreadsheet_id: options[:'drive-key'],
           sheet_ids: options[:'sheets'],

data/lib/ad_localize/repositories/g_sheets_repository.rb

@@ -19,7 +19,7 @@ module AdLocalize
           spreadsheet.sheets[0..0].map { |sheet| get_sheet_values(spreadsheet_id: g_spreadsheet_options.spreadsheet_id, sheet: sheet) }
         else
           spreadsheet.sheets.select do |sheet|
-            g_spreadsheet_options.sheet_ids.include?(sheet.properties.sheet_id)
+            g_spreadsheet_options.sheet_ids.include?(sheet.properties.sheet_id.to_s)
           end.map do |sheet|
             get_sheet_values(spreadsheet_id: g_spreadsheet_options.spreadsheet_id, sheet: sheet)
           end

data/lib/ad_localize/requests/export_request.rb

@@ -3,7 +3,7 @@ module AdLocalize
     class ExportRequest
       SUPPORTED_PLATFORMS = %w(ios android yml json properties).freeze
       DEFAULT_EXPORT_FOLDER = 'exports'.freeze
-      CSV_CONTENT_TYPES = %w(text/csv text/plain).freeze
+      CSV_CONTENT_TYPES = %w(text/csv text/plain application/csv).freeze
 
       def initialize(**args)
         @locales = Array(args[:locales].presence)
@@ -61,7 +61,7 @@ module AdLocalize
       end
 
       def is_csv?(path:)
-        (MIME::Types.of(path).map(&:content_type) & CSV_CONTENT_TYPES).present?
+        CSV_CONTENT_TYPES.include?(`file --brief --mime-type "#{path}"`.strip)
       end
 
       def valid_g_spreadsheet_options?

data/lib/ad_localize/version.rb

@@ -1,3 +1,3 @@
 module AdLocalize
-  VERSION = "4.0.1"
+  VERSION = "4.0.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ad_localize
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 4.0.6
 platform: ruby
 authors:
 - Edouard Siegel
@@ -188,20 +188,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.12'
-- !ruby/object:Gem::Dependency
-  name: mime-types
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
 description: |-
   AdLocalize produces localization files from platform agnostic wording.
                             Supported wording format : CSV. Supported export format: iOS, Android, JSON and YAML