[GitHub] commons-lang issue #224: [LANG-1306] Add nullToEmpty method in StringUtils

2017-01-02 Thread arbasha
Github user arbasha commented on the issue:

https://github.com/apache/commons-lang/pull/224
  
@PascalSchumacher Oops!! Didn't see the existing one 
StringUtils.defaultString(final String str), Nevertheless thanks for reviewing 
the pull request, will close this one. Thanks Much!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] commons-lang issue #224: [LANG-1306] Add nullToEmpty method in StringUtils

2017-01-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue:

https://github.com/apache/commons-lang/pull/224
  
There is also `StringUtils.defaultString(final String str)` which has 
exactly the same functionality.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] commons-lang issue #224: [LANG-1306] Add nullToEmpty method in StringUtils

2017-01-02 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue:

https://github.com/apache/commons-lang/pull/224
  
Thanks for the pull request! :)

The same behavior can be achieved with `ObjectUtils.defaultIfNull(str, 
"")`, but of course this new method reads much better, so I'm in favor of 
merging. Best as on the commons developer list 
(https://commons.apache.org/proper/commons-lang/mail-lists.html) if there are 
any objections.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] commons-lang issue #224: [LANG-1306] Add nullToEmpty method in StringUtils

2016-12-31 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/commons-lang/pull/224
  

[![Coverage 
Status](https://coveralls.io/builds/9478410/badge)](https://coveralls.io/builds/9478410)

Coverage increased (+0.0004%) to 94.363% when pulling 
**cee84c01f986415dd2873d3623f67080d6470c09 on arbasha:master** into 
**021e4dfb501a8e6e5810a4f0efe1267810d62d64 on apache:master**.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---