X-Git-Url: https://nos-oignons.net/gitweb/gestion-adh.git/blobdiff_plain/fcfec54ac07c989195fa9036cee2d5c48eec8337..61077092ef62b9f0780c2dce7ef249be0ce4c7e7:/bin/pre-commit-hook?ds=inline diff --git a/bin/pre-commit-hook b/bin/pre-commit-hook index 317e5c5..921c39c 100755 --- a/bin/pre-commit-hook +++ b/bin/pre-commit-hook @@ -5,10 +5,8 @@ require 'rubygems' require 'bundler' Bundler.setup -require 'safe_yaml' -SafeYAML::OPTIONS[:default_mode] = :safe - -SUBSCRIPTIONS_ROOT = 'Membres' +require 'nos_oignons/git' +require 'nos_oignons/subscriptions' if system('git rev-parse --quiet --verify HEAD >/dev/null') against = 'HEAD' @@ -17,40 +15,15 @@ else against = '4b825dc642cb6eb9a060e54bf8d69288fbee4904' end -def is_valid_subscription?(content) - return false if content.length == 0 - return false unless content.start_with?("---\n") - begin - data = YAML.load(content) - rescue ArgumentError - # Parse error - return false - end - ['name', 'email'].each do |key| - return false unless data.include?(key) - end - true -end - -def is_valid_subscription_file?(file) - IO.popen(['git', 'show', ":#{file}"]) do |f| - is_valid_subscription?(f.read) - end -end - -modified = [] -IO.popen(['git', 'diff-index', '--cached', '--name-status', against]) do |f| - f.readlines.each do |line| - status, file = line.strip.split("\t", 2) - # Has file been added or modified? - if ['A', 'M'].include?(status) - modified << file - next unless file.start_with?("#{SUBSCRIPTIONS_ROOT}/") - if !is_valid_subscription_file?(file) - $stderr.puts "Désolé : #{file} n'a pas le bon format !" - exit 1 - end +IO.popen(['git', 'diff-index', '--cached', '--name-status', against]) do |io| + NosOignons::Git.handle_modified_files(io) do |file| + next unless file.start_with?("#{NosOignons::SUBSCRIPTIONS_ROOT}/") + begin + # Use empty ref to get the index + NosOignons::Subscription.read_from_git('', file) + rescue ArgumentError + $stderr.puts "Désolé : #{file} n'a pas le bon format !" + exit 1 end end end -