lib/cocina/models/mapping/from_mods/part_builder.rb in cocina-models-0.84.4 vs lib/cocina/models/mapping/from_mods/part_builder.rb in cocina-models-0.84.5
- old
+ new
@@ -32,11 +32,11 @@
values = []
values.concat(detail_values.flatten)
values.concat(extent_values.flatten)
values.concat(part_note_value_for(part_element, 'text'))
values.concat(part_note_value_for(part_element, 'date'))
- values.reject!(&:blank?)
+ values.compact_blank!
return if values.empty?
{
type: 'part',
@@ -48,11 +48,11 @@
values = []
values.concat(detail_values)
values.concat(extent_values)
values.concat(part_note_value_for(part_element, 'text'))
values.concat(part_note_value_for(part_element, 'date'))
- values.reject!(&:blank?)
+ values.compact_blank!
return if values.empty?
{
type: 'part',
@@ -81,11 +81,11 @@
def detail_values_for(detail_node)
detail_values = []
detail_values.concat(part_note_value_for(detail_node, 'number'))
detail_values.concat(part_note_value_for(detail_node, 'caption'))
detail_values.concat(part_note_value_for(detail_node, 'title'))
- detail_values.reject!(&:blank?)
+ detail_values.compact_blank!
if detail_values.present?
detail_values.concat(part_note_value_for(detail_node, 'detail type',
xpath: '@type'))
end
detail_values
@@ -99,10 +99,10 @@
def extent_values_for(extent_node)
extent_values = []
extent_values.concat(part_note_value_for(extent_node, 'list'))
extent_values << pages_for(extent_node)
- extent_values.reject!(&:blank?)
+ extent_values.compact_blank!
if extent_values.present?
extent_values.concat(part_note_value_for(extent_node, 'extent unit',
xpath: '@unit'))
end
extent_values