Merge pull request railsadminteam#2109 from quainjn/blank-object-labels

Use default label if object label would be blank

Author: mshibuya

lib/rails_admin/config/model.rb

@@ -44,7 +44,8 @@ def excluded?
       end
 
       def object_label
-        bindings[:object].send object_label_method
+        bindings[:object].send(object_label_method).presence ||
+          bindings[:object].send(:rails_admin_default_object_label_method)
       end
 
       # The display for a model instance (i.e. a single database record).

spec/rails_admin/config/model_spec.rb

@@ -26,6 +26,13 @@
       c = Comment.new(content: 'test')
       expect(RailsAdmin.config(Comment).with(object: c).object_label).to eq('test')
     end
+
+    context 'when the object_label_method is blank' do
+      it 'uses the rails admin default' do
+        c = Comment.create(content: '', id: '1')
+        expect(RailsAdmin.config(Comment).with(object: c).object_label).to eq('Comment #1')
+      end
+    end
   end
 
   describe '#object_label_method' do