Download as
Browse Code »

Commit 24b37d9a0ae70245ee84f738a0d816157433fabf

Authored by Rodrigo Souto
1 parent d2035875
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

[stoa] Filtering leading zeroes from usp_id and cpf 

(ActionItem2293)
Showing 2 changed files with 8 additions and 3 deletions   Show diff stats
plugins/stoa/lib/stoa_plugin/usp_user.rb
  Diff comments   View file @24b37d9
... ... @@ -9,15 +9,15 @@ class StoaPlugin::UspUser < ActiveRecord::Base
9 9 alias_attribute :birth_date, :dtanas
10 10  
11 11 def self.exists?(usp_id)
12   - !StoaPlugin::UspUser.find(:first, :conditions => {:codpes => usp_id}).nil?
  12 + !StoaPlugin::UspUser.find(:first, :conditions => {:codpes => usp_id.to_i}).nil?
13 13 end
14 14  
15 15 def self.matches?(usp_id, field, value)
16   - user = StoaPlugin::UspUser.find(:first, :conditions => {:codpes => usp_id})
  16 + user = StoaPlugin::UspUser.find(:first, :conditions => {:codpes => usp_id.to_i})
17 17 return false if user.nil? || !user.respond_to?(field) || value.blank?
18 18 case field.to_sym
19 19 when :cpf
20   - user.cpf == Digest::MD5.hexdigest(SALT+value.to_s)
  20 + user.cpf == Digest::MD5.hexdigest(SALT+value.to_i.to_s)
21 21 when :birth_date
22 22 user.birth_date.to_s == value
23 23 end
... ...
plugins/stoa/test/unit/usp_user.rb
  Diff comments   View file @24b37d9
... ... @@ -37,5 +37,10 @@ class StoaPlugin::UspUserTest < ActiveSupport::TestCase
37 37 assert !StoaPlugin::UspUser.matches?(123456, :birth_date, '1999-01-30')
38 38 assert !StoaPlugin::UspUser.matches?(654321, :birth_date, '1970-01-30')
39 39 end
  40 +
  41 + should 'filter leading zeroes of id codes on exists and matches' do
  42 + assert StoaPlugin::UspUser.exists?('0000123456')
  43 + assert StoaPlugin::UspUser.matches?(123456, :cpf, '00012345678')
  44 + end
40 45 end
41 46  
... ...