ccls-ccls_engine 3.11.0

data/README.rdoc

@@ -0,0 +1,182 @@
+= CCLS Engine
+
+This is a gem, basic rails app and plugin/engine (GRAPE).
+As a gem, it shares functionality and access to the shared database.
+As an app, it is used to manage the migrations of the shared database.
+It is no longer really used as a plugin, but does contain the
+relevant features of an 'engine' including models, controllers,
+views and routes to the aforementioned resources.
+
+
+== Development Notes
+
+As this code is stored in publicly accessible repositories,
+DO NOT UNDER ANY CIRCUMSTANCES PUT ANY REAL SUBJECT DATA IN THE CODE!
+
+The models in this gem are stored in a shared database, therefore, the migrations
+that create and modify it, as well as the ActiveRecord models that use it must
+reflect this.  Currently, the models which are normally subclasses of ActiveRecord::Base
+need to now by subclasses of ActiveRecordShared.  The migrations, normally subclasses 
+of ActiveRecord::Migration, need to be subclasses of SharedMigration.
+
+
+
+Several errors I get on my MacBook when running rake test:coverage that I don't get running autotest,
+  I also don't get these on my Mac Pro ???
+
+  (See full trace by running task with --trace)
+  jake@mbp-3 : ccls_engine 555> ruby -v
+  ruby 1.8.7 (2010-08-16 patchlevel 302) [i686-darwin9.8.0]
+
+  ArgumentError: wrong number of arguments (1 for 0)
+    app/models/search.rb:183:in `flatten'
+    app/models/search.rb:183:in `having_options'
+    app/models/search.rb:175:in `having'
+    app/models/study_subject_search.rb:46:in `study_subjects'
+
+  ArgumentError: odd number of arguments for Hash
+    app/models/abstract.rb:301:in `[]'
+    app/models/abstract.rb:301:in `comparable_attributes'
+    app/models/abstract.rb:309:in `diff'
+    app/models/abstract.rb:305:in `is_the_same_as?'
+
+  Ccls/Pii should return join of mother's name: .
+  DEPRECATION WARNING: String#chars has been deprecated in favor of String#mb_chars. (called from initials at /Users/jake/github_repo/ccls/ccls_engine/app/models/pii.rb:57)
+
+  test_@@_should_get_new_with_superuser_login(Ccls::RefusalReasonsControllerTest):
+  ActionView::TemplateError: undefined method `find_index' for [:code]:Array
+    On line #4 of app/views/refusal_reasons/_form.html.erb
+
+
+
+===Some general ActiveRecord model notes:
+
+Don't have validations that could raise errors that the user can't do anything about.
+
+When using validates_presence_of, use allow_blank => true on other validations to avoid multiple errors if blank.
+
+When validating the presence of associations, validate on foreign key rather than association so that the error shows up correctly in view.  Unfortunately, this will not guarantee that an association actually exists with that id as it would if the validation was on the association.
+
+ validates_presence_of :project ( will not raise error on project_id so in the view, the project_id selector won't have an error and so won't be highlighted like other errors will be.)
+ validates_presence_of :project_id ( will not ensure that an actual project for the given id exists, but will show the error correctly in the view. )
+
+ I would like to have a validation that ensures that the _id is legitimate.  I do not think that validates_associated is what I want.  Having the both does what I want, but then the view will show "Project is blank" twice.
+
+
+This seems to work ...
+ validates_presence_of :project, :if => :project_id
+ validates_presence_of :project_id
+... but is a bit of a misnomer.  The association validation will raise a 'blank' error, rather than a more appropriate 'non-existant' error, although the message could be altered.
+
+
+Still some models and associations may need done due to priorities and the fact that some never end up in a view and the association may be problematic. (Aliquot,HomexOutcome,Patient)
+
+
+===Some general ActiveSupport::TestCase unit test notes:
+
+Most, if not all, of the 'class level' assertions are defined in ccls-common_lib.
+They call 'create_object', which, unless explicitly defined, is handled in a
+method_missing handler which extracts the model name from the testing class
+and uses the factory of the same name.  (ie. AbstractTest -> Factory(:abstract) )
+In addition, this same method missing handler is used for handling undefined 
+methods like 'create_abstract'.  The handler takes this method which 
+matches /create_(.*)/ and calls a Factory($1)
+
+Also note, create_object only works in unit tests as they are generally associated
+with a particular model.  Controllers, on the other hand, do not share this
+privilege as they are not.  The create_MODEL_NAME technique does still work.
+
+These create_* methods call Factory.build and then save.
+This is predominantly because by default Factory uses create! which raises
+errors rather than returning false which makes testing the error difficult.
+I think that there is a way to set the "default_strategy" for factory_girl,
+but I've yet to figure that out.
+
+
+
+== Required Gem Sources
+
+ gem sources -a http://rubygems.org
+ gem sources -a http://gems.github.com
+
+== Required Gems
+
+* rails ~> 2
+* jrails
+* chronic	#	trying to remove
+* ruby-hmac
+* ssl_requirement
+* ryanb-acts-as-list
+* RedCloth	#	may not use this anymore
+* {ccls-common_lib}[http://github.com/ccls/common_lib]
+* {ccls-calnet_authenticated}[http://github.com/ccls/calnet_authenticated]
+* {ccls-simply_authorized}[http://github.com/ccls/simply_authorized]
+* {ccls-use_db}[http://github.com/ccls/use_db]
+
+== Installation (as a gem/plugin/engine)
+
+In your config/environment.rb
+
+ config.gem 'ccls-ccls_engine'
+
+
+Generate an initializer and copy in some files.
+
+ script/generate ccls_engine
+
+( why don't I copy in these yml files, or at least their examples? )
+
+
+Also need to manually add a shared_database.yml with matching configuration.
+
+ cp config/shared_database.yml.example config/shared_database.yml
+
+
+== Production Installation
+
+Initial database setup (should already be done so don't do this)
+
+ drop the shared production database
+ create the shared production database
+ rake db:migrate RAILS_ENV=production
+
+ # import all the test AND production fixtures
+ rake ccls:full_update RAILS_ENV=production
+
+ # DESTRUCTIVE.  Destroy existing data and parse the csv
+ # files on the "S Drive". These tasks should be
+ # commented out to prevent this from happening.
+ rake odms_import:csv_data
+
+
+
+== Production Updating
+
+ git pull
+ #	No longer uses the Gemfile as it meddled with the gem creation.
+ #	Should link the Gemfile.tmp to Gemfile in home dir.
+ bundle update	
+ rake db:migrate RAILS_ENV=production
+
+
+== Testing
+
+ rake db:migrate
+ rake test and/or autotest
+ 
+
+== Gemified with Jeweler
+
+ vi Rakefile
+ rake version:write
+
+ rake version:bump:patch
+ rake version:bump:minor
+ rake version:bump:major
+
+ rake gemspec
+
+ rake install
+ rake release
+
+Copyright (c) 2010 [Jake Wendt], released under the MIT license

data/app/models/abstract.rb

@@ -0,0 +1,181 @@
+#	Abstract model
+class Abstract < ActiveRecordShared
+
+	belongs_to :study_subject, :counter_cache => true
+
+	with_options :class_name => 'User', :primary_key => 'uid' do |u|
+		u.belongs_to :entry_1_by, :foreign_key => 'entry_1_by_uid'
+		u.belongs_to :entry_2_by, :foreign_key => 'entry_2_by_uid'
+		u.belongs_to :merged_by,  :foreign_key => 'merged_by_uid'
+	end
+
+	include AbstractValidations
+
+	attr_protected :study_subject_id, :study_subject
+	attr_protected :entry_1_by_uid
+	attr_protected :entry_2_by_uid
+	attr_protected :merged_by_uid
+
+	attr_accessor :current_user
+	attr_accessor :weight_units, :height_units
+	attr_accessor :merging	#	flag to be used to skip 2 abstract limitation
+
+	#	The :on => :create doesn't seem to work as described
+	#	validate_on_create is technically deprecated, but still works
+	validate_on_create :subject_has_less_than_three_abstracts	#, :on => :create
+	validate_on_create :subject_has_no_merged_abstract	#, :on => :create
+
+	before_create :set_user
+	after_create  :delete_unmerged
+	before_save   :convert_height_to_cm
+	before_save   :convert_weight_to_kg
+	before_save   :set_days_since_fields
+
+	def self.fields
+		#	db: db field name
+		#	human: humanized field
+		@@fields ||= YAML::load( ERB.new( IO.read(
+			File.join(File.dirname(__FILE__),'../../config/abstract_fields.yml')
+		)).result)
+	end
+
+	def fields
+		Abstract.fields
+	end
+
+	def self.db_fields
+#		@db_fields ||= fields.collect{|f|f[:db]}
+		Abstract.fields.collect{|f|f[:db]}
+	end
+
+	def db_fields
+		Abstract.db_fields
+	end
+
+	def comparable_attributes
+		HashWithIndifferentAccess[attributes.select {|k,v| db_fields.include?(k)}]
+	end
+
+	def is_the_same_as?(another_abstract)
+		self.diff(another_abstract).blank?
+	end
+
+	def diff(another_abstract)
+		a1 = self.comparable_attributes
+		a2 = Abstract.find(another_abstract).comparable_attributes
+		HashWithIndifferentAccess[a1.select{|k,v| a2[k] != v unless( a2[k].blank? && v.blank? ) }]
+	end
+
+#	include AbstractSearch
+	def self.search(params={})
+		#	TODO	stop using this.  Now that study subjects and abstracts are in
+		#		the same database, this should be simplified.  Particularly since
+		#		the only searching is really on the study subject and not the abstract.
+		AbstractSearch.new(params).abstracts
+	end
+
+	def self.sections
+		#	:label: Cytogenetics
+		#	:controller: CytogeneticsController
+		#	:edit:  :edit_abstract_cytogenetic_path
+		#	:show:  :abstract_cytogenetic_path
+		@@sections ||= YAML::load(ERB.new( IO.read(
+			File.join(File.dirname(__FILE__),'../../config/abstract_sections.yml')
+		)).result)
+	end
+
+	def merged?
+		!merged_by_uid.blank?
+	end
+
+protected
+
+	def set_days_since_fields
+		#	must explicitly convert these DateTimes to Date so that the
+		#	difference is in days and not seconds
+		#	I really only need to do this if something changes,
+		#	but for now, just do it and make sure that
+		#	it is tested.  Optimize and refactor later.
+		unless diagnosed_on.nil?
+			self.response_day_7_days_since_diagnosis = (
+				response_report_on_day_7.to_date - diagnosed_on.to_date 
+				) unless response_report_on_day_7.nil?
+			self.response_day_14_days_since_diagnosis = (
+				response_report_on_day_14.to_date - diagnosed_on.to_date 
+				) unless response_report_on_day_14.nil?
+			self.response_day_28_days_since_diagnosis = (
+				response_report_on_day_28.to_date - diagnosed_on.to_date 
+				) unless response_report_on_day_28.nil?
+		end
+		unless treatment_began_on.nil?
+			self.response_day_7_days_since_treatment_began = (
+				response_report_on_day_7.to_date - treatment_began_on.to_date 
+				) unless response_report_on_day_7.nil?
+			self.response_day_14_days_since_treatment_began = (
+				response_report_on_day_14.to_date - treatment_began_on.to_date 
+				) unless response_report_on_day_14.nil?
+			self.response_day_28_days_since_treatment_began = (
+				response_report_on_day_28.to_date - treatment_began_on.to_date 
+				) unless response_report_on_day_28.nil?
+		end
+	end
+
+	def convert_height_to_cm
+		if( !height_units.nil? && height_units.match(/in/i) )
+			self.height_units = nil
+			self.height_at_diagnosis *= 2.54
+		end
+	end
+
+	def convert_weight_to_kg
+		if( !weight_units.nil? && weight_units.match(/lb/i) )
+			self.weight_units = nil
+			self.weight_at_diagnosis /= 2.2046
+		end
+	end
+
+	#	Set user if given
+	def set_user
+		if study_subject
+			#	because it is possible to create the first, then the second
+			#	and then delete the first, and create another, first and
+			#	second kinda lose their meaning until the merge, so set them
+			#	both as the same until the merge
+			case study_subject.abstracts_count
+				when 0 
+					self.entry_1_by_uid = current_user.try(:uid)||0
+					self.entry_2_by_uid = current_user.try(:uid)||0
+				when 1 
+					self.entry_1_by_uid = current_user.try(:uid)||0
+					self.entry_2_by_uid = current_user.try(:uid)||0
+				when 2
+					abs = study_subject.abstracts
+					#	compact just in case a nil crept in
+					self.entry_1_by_uid = [abs[0].entry_1_by_uid,abs[0].entry_2_by_uid].compact.first
+					self.entry_2_by_uid = [abs[1].entry_1_by_uid,abs[1].entry_2_by_uid].compact.first
+					self.merged_by_uid  = current_user.try(:uid)||0
+			end
+		end
+	end
+
+	def delete_unmerged
+		if study_subject and !merged_by_uid.blank?
+			#	use delete and not destroy to preserve the abstracts_count
+			study_subject.unmerged_abstracts.each{|a|a.delete}
+		end
+	end
+
+	def subject_has_less_than_three_abstracts
+		#	because this abstract hasn't been created yet, we're comparing to 2, not 3
+		if study_subject and study_subject.abstracts_count >= 2
+			errors.add(:study_subject_id,"Study Subject can only have 2 abstracts." ) unless merging
+		end
+	end
+
+	def subject_has_no_merged_abstract
+		if study_subject and study_subject.merged_abstract
+			errors.add(:study_subject_id,"Study Subject already has a merged abstract." )
+		end
+	end
+
+end

data/app/models/abstract_search.rb

@@ -0,0 +1,50 @@
+
+#	This is only used in the Abstract app and only seems to use :merged
+#	Everything else is unused.
+
+class AbstractSearch < Search
+
+	self.searchable_attributes += [ :q, :merged ]
+
+	self.valid_orders = self.valid_orders.merge({
+		:id => 'abstracts.id'
+	})
+
+	def abstracts
+		@abstracts ||= Abstract.send(
+			(paginate?)?'paginate':'all',{
+				:order => search_order,
+				:joins => joins,
+				:conditions => conditions
+			}.merge(
+				(paginate?)?{
+					:per_page => per_page||25,
+					:page     => page||1
+				}:{}
+			)
+		)
+	end
+
+private	#	THIS IS REQUIRED
+
+	def merged_conditions
+#	TODO what if merged is false?
+		['abstracts.merged_by_uid IS NOT NULL'] unless merged.blank?
+	end
+
+	#	Certainly not the fastest way, but quite possibly the only way
+	#	to search against study_subjects being in a separate database.
+#
+#	TODO StudySubjects are now in the same database as the Abstracts so
+#	this unnecessarily complicated searching should be removed.
+#
+	def study_subjects_conditions
+		unless q.blank?
+			study_subjects = StudySubject.search(:q => q, :paginate => false)
+			study_subject_ids = study_subjects.collect(&:id)
+			['abstracts.study_subject_id IN (:study_subject_ids)', 
+				{ :study_subject_ids => study_subject_ids } ]
+		end
+	end
+
+end

data/app/models/abstract_validations.rb

@@ -0,0 +1,324 @@
+#
+#	Simply extracted some code to clean up model.
+#	I'd like to do this to all of the really big classes
+#	but let's see how this goes first.
+#
+module AbstractValidations
+def self.included(base)
+#	Must delay the calls to these ActiveRecord methods
+#	or it will raise many "undefined method"s.
+base.class_eval do
+
+	with_options :allow_blank => true do |b|
+		b.with_options :maximum => 2 do |o|
+			o.validates_length_of( :response_classification_day_14 )
+			o.validates_length_of( :response_classification_day_28 )
+			o.validates_length_of( :response_classification_day_7 )
+		end
+		b.with_options :maximum => 3 do |o|
+			o.validates_length_of( :cytogen_chromosome_number )
+		end
+		b.with_options :maximum => 4 do |o|
+			o.validates_length_of( :flow_cyto_other_marker_1 )
+			o.validates_length_of( :flow_cyto_other_marker_2 )
+			o.validates_length_of( :flow_cyto_other_marker_3 )
+			o.validates_length_of( :flow_cyto_other_marker_4 )
+			o.validates_length_of( :flow_cyto_other_marker_5 )
+			o.validates_length_of( :response_other1_value_day_14 )
+			o.validates_length_of( :response_other1_value_day_7 )
+			o.validates_length_of( :response_other2_value_day_14 )
+			o.validates_length_of( :response_other2_value_day_7 )
+			o.validates_length_of( :response_other3_value_day_14 )
+			o.validates_length_of( :response_other4_value_day_14 )
+			o.validates_length_of( :response_other5_value_day_14 )
+		end
+		b.with_options :maximum => 5 do |o|
+			o.validates_length_of( :normal_cytogen )
+			o.validates_length_of( :is_cytogen_hosp_fish_t1221_done )
+			o.validates_length_of( :is_karyotype_normal )
+			o.validates_length_of( :physical_neuro )
+			o.validates_length_of( :physical_other_soft_2 )
+			o.validates_length_of( :physical_gingival )
+			o.validates_length_of( :physical_leukemic_skin )
+			o.validates_length_of( :physical_lymph )
+			o.validates_length_of( :physical_spleen )
+			o.validates_length_of( :physical_testicle )
+			o.validates_length_of( :physical_other_soft )
+			o.validates_length_of( :is_hypodiploid )
+			o.validates_length_of( :is_hyperdiploid )
+			o.validates_length_of( :is_diploid )
+			o.validates_length_of( :dna_index )
+			o.validates_length_of( :cytogen_is_hyperdiploidy )
+		end
+		b.with_options :maximum => 9 do |o|
+			o.validates_length_of( :cytogen_t1221 )
+			o.validates_length_of( :cytogen_inv16 )
+			o.validates_length_of( :cytogen_t119 )
+			o.validates_length_of( :cytogen_t821 )
+			o.validates_length_of( :cytogen_t1517 )
+		end
+		b.with_options :maximum => 10 do |o|
+			o.validates_length_of( :response_cd10_day_14 )
+			o.validates_length_of( :response_cd10_day_7 )
+			o.validates_length_of( :response_cd13_day_14 )
+			o.validates_length_of( :response_cd13_day_7 )
+			o.validates_length_of( :response_cd14_day_14 )
+			o.validates_length_of( :response_cd14_day_7 )
+			o.validates_length_of( :response_cd15_day_14 )
+			o.validates_length_of( :response_cd15_day_7 )
+			o.validates_length_of( :response_cd19_day_14 )
+			o.validates_length_of( :response_cd19_day_7 )
+			o.validates_length_of( :response_cd19cd10_day_14 )
+			o.validates_length_of( :response_cd19cd10_day_7 )
+			o.validates_length_of( :response_cd1a_day_14 )
+			o.validates_length_of( :response_cd2a_day_14 )
+			o.validates_length_of( :response_cd20_day_14 )
+			o.validates_length_of( :response_cd20_day_7 )
+			o.validates_length_of( :response_cd3a_day_14 )
+			o.validates_length_of( :response_cd3_day_7 )
+			o.validates_length_of( :response_cd33_day_14 )
+			o.validates_length_of( :response_cd33_day_7 )
+			o.validates_length_of( :response_cd34_day_14 )
+			o.validates_length_of( :response_cd34_day_7 )
+			o.validates_length_of( :response_cd4a_day_14 )
+			o.validates_length_of( :response_cd5a_day_14 )
+			o.validates_length_of( :response_cd56_day_14 )
+			o.validates_length_of( :response_cd61_day_14 )
+			o.validates_length_of( :response_cd7a_day_14 )
+			o.validates_length_of( :response_cd8a_day_14 )
+			o.validates_length_of( :flow_cyto_cd10 )
+			o.validates_length_of( :flow_cyto_igm )
+			o.validates_length_of( :flow_cyto_bm_kappa )
+			o.validates_length_of( :flow_cyto_bm_lambda )
+			o.validates_length_of( :flow_cyto_cd10_19 )
+			o.validates_length_of( :flow_cyto_cd19 )
+			o.validates_length_of( :flow_cyto_cd20 )
+			o.validates_length_of( :flow_cyto_cd21 )
+			o.validates_length_of( :flow_cyto_cd22 )
+			o.validates_length_of( :flow_cyto_cd23 )
+			o.validates_length_of( :flow_cyto_cd24 )
+			o.validates_length_of( :flow_cyto_cd40 )
+			o.validates_length_of( :flow_cyto_surface_ig )
+			o.validates_length_of( :flow_cyto_cd1a )
+			o.validates_length_of( :flow_cyto_cd2 )
+			o.validates_length_of( :flow_cyto_cd3 )
+			o.validates_length_of( :flow_cyto_cd4 )
+			o.validates_length_of( :flow_cyto_cd5 )
+			o.validates_length_of( :flow_cyto_cd7 )
+			o.validates_length_of( :flow_cyto_cd8 )
+			o.validates_length_of( :flow_cyto_cd3_cd4 )
+			o.validates_length_of( :flow_cyto_cd3_cd8 )
+			o.validates_length_of( :flow_cyto_cd11b )
+			o.validates_length_of( :flow_cyto_cd11c )
+			o.validates_length_of( :flow_cyto_cd13 )
+			o.validates_length_of( :flow_cyto_cd15 )
+			o.validates_length_of( :flow_cyto_cd33 )
+			o.validates_length_of( :flow_cyto_cd41 )
+			o.validates_length_of( :flow_cyto_cdw65 )
+			o.validates_length_of( :flow_cyto_cd34 )
+			o.validates_length_of( :flow_cyto_cd61 )
+			o.validates_length_of( :flow_cyto_cd14 )
+			o.validates_length_of( :flow_cyto_glycoa )
+			o.validates_length_of( :flow_cyto_cd16 )
+			o.validates_length_of( :flow_cyto_cd56 )
+			o.validates_length_of( :flow_cyto_cd57 )
+			o.validates_length_of( :flow_cyto_cd9 )
+			o.validates_length_of( :flow_cyto_cd25 )
+			o.validates_length_of( :flow_cyto_cd38 )
+			o.validates_length_of( :flow_cyto_cd45 )
+			o.validates_length_of( :flow_cyto_cd71 )
+			o.validates_length_of( :flow_cyto_tdt )
+			o.validates_length_of( :flow_cyto_hladr )
+			o.validates_length_of( :response_hladr_day_14 )
+			o.validates_length_of( :response_hladr_day_7 )
+			o.validates_length_of( :response_tdt_day_14 )
+			o.validates_length_of( :response_tdt_day_7 )
+		end
+		b.with_options :maximum => 15 do |o|
+			o.validates_length_of( :response_blasts_units_day_14 )
+			o.validates_length_of( :response_blasts_units_day_28 )
+			o.validates_length_of( :response_blasts_units_day_7 )
+			o.validates_length_of( :other_dna_measure )
+			o.validates_length_of( :response_fab_subtype )
+		end
+		b.with_options :maximum => 20 do |o|
+			o.validates_length_of( :flow_cyto_other_marker_1_name )
+			o.validates_length_of( :flow_cyto_other_marker_2_name )
+			o.validates_length_of( :flow_cyto_other_marker_3_name )
+			o.validates_length_of( :flow_cyto_other_marker_4_name )
+			o.validates_length_of( :flow_cyto_other_marker_5_name )
+		end
+		b.with_options :maximum => 25 do |o|
+			o.validates_length_of( :response_other1_name_day_14 )
+			o.validates_length_of( :response_other1_name_day_7 )
+			o.validates_length_of( :response_other2_name_day_14 )
+			o.validates_length_of( :response_other2_name_day_7 )
+			o.validates_length_of( :response_other3_name_day_14 )
+			o.validates_length_of( :response_other4_name_day_14 )
+			o.validates_length_of( :response_other5_name_day_14 )
+		end
+		b.with_options :maximum => 35 do |o|
+			o.validates_length_of( :cytogen_other_trans_1 )
+			o.validates_length_of( :cytogen_other_trans_2 )
+			o.validates_length_of( :cytogen_other_trans_3 )
+			o.validates_length_of( :cytogen_other_trans_4 )
+			o.validates_length_of( :cytogen_other_trans_5 )
+			o.validates_length_of( :cytogen_other_trans_6 )
+			o.validates_length_of( :cytogen_other_trans_7 )
+			o.validates_length_of( :cytogen_other_trans_8 )
+			o.validates_length_of( :cytogen_other_trans_9 )
+			o.validates_length_of( :cytogen_other_trans_10 )
+		end
+		b.with_options :maximum => 50 do |o|
+			o.validates_length_of( :flow_cyto_igm_text )
+			o.validates_length_of( :flow_cyto_bm_kappa_text )
+			o.validates_length_of( :flow_cyto_bm_lambda_text )
+			o.validates_length_of( :flow_cyto_cd10_19_text )
+			o.validates_length_of( :flow_cyto_cd10_text )
+			o.validates_length_of( :flow_cyto_cd19_text )
+			o.validates_length_of( :flow_cyto_cd20_text )
+			o.validates_length_of( :flow_cyto_cd21_text )
+			o.validates_length_of( :flow_cyto_cd22_text )
+			o.validates_length_of( :flow_cyto_cd23_text )
+			o.validates_length_of( :flow_cyto_cd24_text )
+			o.validates_length_of( :flow_cyto_cd40_text )
+			o.validates_length_of( :flow_cyto_surface_ig_text )
+			o.validates_length_of( :flow_cyto_cd1a_text )
+			o.validates_length_of( :flow_cyto_cd2_text )
+			o.validates_length_of( :flow_cyto_cd3_text )
+			o.validates_length_of( :flow_cyto_cd4_text )
+			o.validates_length_of( :flow_cyto_cd5_text )
+			o.validates_length_of( :flow_cyto_cd7_text )
+			o.validates_length_of( :flow_cyto_cd8_text )
+			o.validates_length_of( :flow_cyto_cd3_cd4_text )
+			o.validates_length_of( :flow_cyto_cd3_cd8_text )
+			o.validates_length_of( :flow_cyto_cd11b_text )
+			o.validates_length_of( :flow_cyto_cd11c_text )
+			o.validates_length_of( :flow_cyto_cd13_text )
+			o.validates_length_of( :flow_cyto_cd15_text )
+			o.validates_length_of( :flow_cyto_cd33_text )
+			o.validates_length_of( :flow_cyto_cd41_text )
+			o.validates_length_of( :flow_cyto_cdw65_text )
+			o.validates_length_of( :flow_cyto_cd34_text )
+			o.validates_length_of( :flow_cyto_cd61_text )
+			o.validates_length_of( :flow_cyto_cd14_text )
+			o.validates_length_of( :flow_cyto_glycoa_text )
+			o.validates_length_of( :flow_cyto_cd16_text )
+			o.validates_length_of( :flow_cyto_cd56_text )
+			o.validates_length_of( :flow_cyto_cd57_text )
+			o.validates_length_of( :flow_cyto_cd9_text )
+			o.validates_length_of( :flow_cyto_cd25_text )
+			o.validates_length_of( :flow_cyto_cd38_text )
+			o.validates_length_of( :flow_cyto_cd45_text )
+			o.validates_length_of( :flow_cyto_cd71_text )
+			o.validates_length_of( :flow_cyto_tdt_text )
+			o.validates_length_of( :flow_cyto_hladr_text )
+			o.validates_length_of( :flow_cyto_other_marker_1_text )
+			o.validates_length_of( :flow_cyto_other_marker_2_text )
+			o.validates_length_of( :flow_cyto_other_marker_3_text )
+			o.validates_length_of( :flow_cyto_other_marker_4_text )
+			o.validates_length_of( :flow_cyto_other_marker_5_text )
+			o.validates_length_of( :ucb_fish_results )
+			o.validates_length_of( :fab_classification )
+			o.validates_length_of( :diagnosis_icdo_number )
+			o.validates_length_of( :cytogen_t922 )
+		end
+		b.with_options :maximum => 55 do |o|
+			o.validates_length_of( :diagnosis_icdo_description )
+		end
+		b.with_options :maximum => 100 do |o|
+			o.validates_length_of( :ploidy_comment )
+		end
+		b.with_options :maximum => 250 do |o|
+			o.validates_length_of( :csf_red_blood_count_text )
+			o.validates_length_of( :blasts_are_present )
+			o.validates_length_of( :peripheral_blood_in_csf )
+			o.validates_length_of( :chemo_protocol_report_found )
+			o.validates_length_of( :chemo_protocol_name )
+			o.validates_length_of( :conventional_karyotype_results )
+			o.validates_length_of( :hospital_fish_results )
+			o.validates_length_of( :hyperdiploidy_by )
+		end
+		b.with_options :maximum => 65000 do |o|
+			o.validates_length_of( :marrow_biopsy_diagnosis )
+			o.validates_length_of( :marrow_aspirate_diagnosis )
+			o.validates_length_of( :csf_white_blood_count_text )
+			o.validates_length_of( :csf_comment )
+			o.validates_length_of( :chemo_protocol_agent_description )
+			o.validates_length_of( :chest_imaging_comment )
+			o.validates_length_of( :cytogen_comment )
+			o.validates_length_of( :discharge_summary )
+			o.validates_length_of( :flow_cyto_remarks )
+			o.validates_length_of( :response_comment_day_7 )
+			o.validates_length_of( :response_comment_day_14 )
+			o.validates_length_of( :histo_report_results )
+			o.validates_length_of( :response_comment )
+		end
+	end
+
+	validates_inclusion_of(
+		:cbc_report_found,
+		:cerebrospinal_fluid_report_found,
+		:chemo_protocol_report_found,
+		:chest_ct_medmass_present,
+		:chest_imaging_report_found,
+		:cytogen_hospital_fish_done,
+		:cytogen_karyotype_done,
+		:cytogen_report_found,
+		:cytogen_trisomy4,
+		:cytogen_trisomy5,
+		:cytogen_trisomy10,
+		:cytogen_trisomy17,
+		:cytogen_trisomy21,
+		:cytogen_ucb_fish_done,
+		:diagnosis_is_all,
+		:diagnosis_is_aml,
+		:diagnosis_is_b_all,
+		:diagnosis_is_cll,
+		:diagnosis_is_cml,
+		:diagnosis_is_other,
+		:diagnosis_is_t_all,
+		:discharge_summary_found,
+		:flow_cyto_num_results_available,
+		:flow_cyto_report_found,
+		:h_and_p_reports_found,
+		:hepatomegaly_present,
+		:histo_report_found,
+		:is_down_syndrome_phenotype,
+		:marrow_aspirate_report_found,
+		:marrow_biopsy_report_found,
+		:mediastinal_mass_present,
+		:patient_on_chemo_protocol,
+		:ploidy_report_found,
+		:received_bone_marrow_aspirate,
+		:received_bone_marrow_biopsy,
+		:received_cbc,
+		:received_chemo_protocol,
+		:received_chest_ct,
+		:received_chest_xray,
+		:received_csf,
+		:received_cytogenetics,
+		:received_discharge_summary,
+		:received_flow_cytometry,
+		:received_hla_typing,
+		:received_h_and_p,
+		:received_other_reports,
+		:received_ploidy,
+		:received_resp_to_therapy,
+		:response_day14or28_flag,
+		:response_day30_is_in_remission,
+		:response_is_inconclusive_day_7,
+		:response_is_inconclusive_day_14,
+		:response_is_inconclusive_day_21,
+		:response_is_inconclusive_day_28,
+		:response_report_found_day_7,
+		:response_report_found_day_14,
+		:response_report_found_day_28,
+		:splenomegaly_present,
+		:tdt_often_found_flow_cytometry,
+		:tdt_report_found,
+			:in => YNDK.valid_values, :allow_nil => true )
+
+end	#	class_eval
+end	#	included
+end	#	AbstractValidations