From: richard.schneeman@... Date: 2016-04-21T15:29:27+00:00 Subject: [ruby-core:75070] [Ruby trunk Feature#12306] Implement String #blank? #present? and improve #strip and family to handle unicode Issue #12306 has been updated by Richard Schneeman. I think this is useful outside of Rails. The Active Support module has 87 million downloads on rubygems.org while Railties has only 53 million downloads. So 34 million times people have needed to use Active Support without Railties, this is a huge number. Granted not all of them will only be for `present?` but some of them will be. In many cases people will want this logic and maybe they cannot or do not wish to pull in Active Support. I find it quite common to either check for presence of a variable and then to see if it is `empty?` or to `strip.empty?`. Here is a quick grep of the examples on my computer (it is by no means comprehensive). ## Nil and empty? checks together ./bundler/lib/bundler/cli/lock.rb: if gems && !gems.empty? ./bundler/lib/bundler/cli/open.rb: editor = [ENV["BUNDLER_EDITOR"], ENV["VISUAL"], ENV["EDITOR"]].find {|e| !e.nil? && !e.empty? } ./bundler/lib/bundler/cli.rb: if ENV["RUBYGEMS_GEMDEPS"] && !ENV["RUBYGEMS_GEMDEPS"].empty? ./carrierwave/lib/carrierwave/uploader/download.rb: return match[1] unless match.nil? || match[1].empty? ./fog/lib/fog/clodo/core.rb: :path => (uri.path and not uri.path.empty?) ? uri.path : 'v1.0'./fog/lib/fog/clodo/models/compute/server.rb: pubaddrs && !pubaddrs.empty? ? pubaddrs.first['ip'] : nil] ./fog/lib/fog/cloudstack/models/compute/servers.rb: if servers.nil? || servers.empty? ./fog/lib/fog/cloudstack/models/compute/servers.rb: unless servers.nil? || servers.empty? ./git_hub_bub/lib/git_hub_bub/request.rb: self.options[:query] = query if query && !query.empty? ./puma/lib/puma/rack/urlmap.rb: next unless !rest || rest.empty? || rest[0] == ?/ ./rack/lib/rack/mock.rb: env[PATH_INFO] = (!uri.path || uri.path.empty?) ? "/" : uri.path ./rack/lib/rack/multipart/parser.rb: elsif !filename && data.empty? ./rack/lib/rack/multipart/parser.rb: if name.nil? || name.empty? ./rack/lib/rack/multipart/parser.rb: if content.nil? || content.empty? ./rack/lib/rack/server.rb: options[:config] = args.last if args.last && !args.last.empty? ./rack/lib/rack/session/abstract/id.rb: value && !value.empty? ./rake/lib/rake/task.rb: add_comment(comment) if comment && ! comment.empty? ./rdoc/lib/rdoc/code_object.rb: if comment and not comment.empty? then ./rdoc/lib/rdoc/parser/c.rb: if no_match and no_match.empty? then ./rest-client/lib/restclient/request.rb: if (!body) || body.empty? ./rpm/lib/new_relic/agent/agent.rb: if data && !data.empty? ./rpm/lib/new_relic/agent/javascript_instrumentor.rb: value.nil? || value.empty? ./rpm/lib/new_relic/agent/obfuscator.rb: if key.nil? || key.empty? ./rubinius/library/rubygems/command.rb: if args.nil? or args.empty? then ./ruby/.ext/common/psych/class_loader.rb: return nil if !klassname || klassname.empty? ./ruby/.ext/common/tk/menu.rb: configure(keys) if keys && !keys.empty? ./ruby/.ext/common/tkextlib/tktable/tktable.rb: obj.configure(keys) if keys && ! keys.empty? ./ruby/.ext/common/tkextlib/tktable/tktable.rb: configure(keys) if keys && ! keys.empty? ./ruby/lib/optparse.rb: elsif !opt or opt.empty? ./ruby/lib/rubygems/command.rb: if args.nil? or args.empty? then ./ruby/lib/rubygems/command.rb: return if option_list.nil? or option_list.empty? ./sass/lib/sass/engine.rb: if content.first && content.first.strip.empty? ./sprockets/lib/sprockets/loader.rb: if cached_asset[:metadata][:included] && !cached_asset[:metadata][:included].empty? ./sprockets/lib/sprockets/loader.rb: if cached_asset[:metadata][:links] && !cached_asset[:metadata][:links].empty? ./sprockets/lib/sprockets/loader.rb: if cached_asset[:metadata][:stubbed] && !cached_asset[:metadata][:stubbed].empty? ./sprockets/lib/sprockets/loader.rb: if cached_asset[:metadata][:required] && !cached_asset[:metadata][:required].empty? ./sprockets/lib/sprockets/loader.rb: if cached_asset[:metadata][:dependencies] && !cached_asset[:metadata][:dependencies].empty? ./www.ruby-lang.org/_plugins/posted_by.rb: if author.nil? || author.empty? || author == 'Unknown Author' ./yard/lib/yard/cli/yri.rb: if @name.nil? || @name.strip.empty? ./yard/lib/yard/parser/ruby/ruby_parser.rb: if comment && !comment.empty? ## strip.empty? checks ./concurrent-ruby/doc/actor/format.rb: unless chunk.strip.empty? || chunk =~ /\A *#/ ./concurrent-ruby/examples/format.rb: unless chunk.strip.empty? || chunk =~ /\A *#/ ./fog/lib/fog/rackspace/service.rb: !response.body.strip.empty? && ./passenger/lib/phusion_passenger/platform_info/compiler.rb: if source.strip.empty? ./passenger/lib/phusion_passenger/platform_info.rb: indent = str.split("\n").select{ |line| !line.strip.empty? }.map{ |line| line.index(/[^\s]/) }.compact.min || 0 ./rdoc/lib/rdoc/context.rb: known.value.nil? or known.value.strip.empty? ./rpm/lib/new_relic/cli/commands/deployments.rb: @description = nil if @description && @description.strip.empty? ./rubinius/rakelib/instruction_parser.rb: elsif line.strip.empty? ./rubinius/tools/rubuildius/bin/pastie.rb: return if body.strip.empty? ./rubinius/vm/codegen/field_extract.rb: return '' if out.strip.empty? ./ruby/ext/ripper/tools/strip.rb: if line.strip.empty? ./ruby/ext/tk/extconf.rb: defs.map{|ary| s = ary.join(''); (s.strip.empty?)? "": "-D" << s} ./ruby/ext/tk/extconf.rb: !TkConfig_Info['TK_XINCLUDES'].strip.empty?) || ./ruby/ext/tk/extconf.rb: (TkConfig_Info['TK_XLIBSW'] && !TkConfig_Info['TK_XLIBSW'].strip.empty?) ./ruby/ext/tk/extconf.rb: !TkConfig_Info['TK_XINCLUDES'].strip.empty? ./ruby/ext/tk/extconf.rb: TkConfig_Info['TK_XLIBSW'] && !TkConfig_Info['TK_XLIBSW'].strip.empty? ./ruby/ext/tk/extconf.rb: !TclConfig_Info['TCL_STUB_LIB_SPEC'].strip.empty? && ./ruby/ext/tk/extconf.rb: !TkConfig_Info['TK_STUB_LIB_SPEC'].strip.empty? ./ruby/ext/tk/extconf.rb: !TclConfig_Info['TCL_BUILD_STUB_LIB_SPEC'].strip.empty? ./ruby/ext/tk/extconf.rb: !TclConfig_Info['TCL_BUILD_LIB_SPEC'].strip.empty? ./ruby/ext/tk/extconf.rb: !TclConfig_Info['TK_BUILD_STUB_LIB_SPEC'].strip.empty? ./ruby/ext/tk/extconf.rb: !TclConfig_Info['TK_BUILD_LIB_SPEC'].strip.empty? ./ruby/lib/net/http/header.rb: .reject {|str| str.strip.empty? }\ ./ruby/lib/rdoc/context.rb: known.value.nil? or known.value.strip.empty? ./sass/lib/sass/engine.rb: if line.strip.empty? ./sass/lib/sass/engine.rb: if value.strip.empty? ./sass/lib/sass/engine.rb: if value.strip.empty? && line.children.empty? ./sass/lib/sass/engine.rb: if content.first && content.first.strip.empty? ./yard/lib/yard/cli/yri.rb: if @name.nil? || @name.strip.empty? ./yard/lib/yard/i18n/text.rb: if line.strip.empty? ./yard/lib/yard/tags/directives.rb: (tag.text && !tag.text.strip.empty?) ./yard/templates/default/docstring/setup.rb: if text.strip.empty? && object.tags(:return).size == 1 && object.tag(:return).text In many of these examples the desired result is not clear at a glance. It would simplify code and comprehension to have a common short hand that can be used when it is not appropriate to pull in Active Support. ---------------------------------------- Feature #12306: Implement String #blank? #present? and improve #strip and family to handle unicode https://bugs.ruby-lang.org/issues/12306#change-58194 * Author: Sam Saffron * Status: Open * Priority: Normal * Assignee: ---------------------------------------- Time and again there have been rejected feature requests to Ruby core to implement `blank` and `present` protocols across all objects as ActiveSupport does. I am fine with this call and think it is fair. However, for the narrow case of String having `#blank?` and `#present?` makes sense. - Provides a natural extension over `#strip`, `#lstrip` and `#rstrip`. `(" ".strip.length == 0) == " ".blank?` - Plays nicely with ActiveSupport, providing an efficient implementation in Ruby core: see: https://github.com/SamSaffron/fast_blank, implementing blank efficiently requires a c extension. However, if this work is to be done, `#strip` and should probably start dealing with unicode blanks, eg: ``` irb(main):008:0> [0x3000].pack("U") => "���" irb(main):009:0> [0x3000].pack("U").strip.length => 1 ``` So there are 2 questions / feature requests here 1. Can we add blank? and present? to String? 2. Can we amend strip and family to account for unicode per: https://github.com/SamSaffron/fast_blank/blob/master/ext/fast_blank/fast_blank.c#L43-L74 -- https://bugs.ruby-lang.org/ Unsubscribe: