OldTruckDriver opened a new pull request, #612:
URL: https://github.com/apache/commons-csv/pull/612
[CSV-328] Fix quoted null string after disabling quote
CSVFormat.Builder#setNullString(String) rebuilt quotedNullString by
concatenating the nullable quoteCharacter field directly. When setQuote(null)
was called before setNullString("NULL"), QuoteMode.ALL printed null as
nullNULLnull instead of "NULL".
This change makes setNullString(String) and setQuote(Character) use the same
quoted-null-string update logic, preserving the existing fallback to the
default double quote when the quote character is disabled.
Tests cover both setter orders for setNullString("NULL") and setQuote(null).
Tests run:
- mvn -q
-Dtest=org.apache.commons.csv.CSVFormatTest#testQuotedNullStringTracksQuoteCharacter
test
- mvn -q -Dtest=org.apache.commons.csv.CSVFormatTest test
- mvn -q
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
