[New Plugin] CSV Lint

PeterJones

My guess that the problem isn’t with the plugin not aligning things. The problem is that you don’t have your encoding correct, so Notepad++ is displaying the n-byte sequence for unicode characters as multiple characters instead of properly displaying the single character represented by those multiple bytes. You might try Settings > Preferences > New Document > Encoding > UTF-8: Apply to opened ANSI files

… which should make Notepad++ use UTF-8 instead of mis-reading that as an Win-1252 file even though it should be UTF-8.

–
Addendum: specifically, Ã¤ are the Win-1252 interpretation of the two bytes at 0xC3 (decimal 195) and 0xA4 (decimal 164). The sequence 0xC3 0xA4 is the two-byte UTF-8 sequence for ä. Once you load a file and it’s misinterpreted the UTF-8 as Win-1252, the easiest way to fix things is to change from Encoding > ANSI to Encoding > UTF-8 (use the top half of the menu, not the bottom “convert” half, because you don’t want to convert the file to a different byte sequence, you just want to change how Notepad++ interprets the bytes it’s already read)

_ C64 CUSTOMS _

@peterjones Ah, great, thanks a lot. I have set the preferences as shown above by you but it somehow hasn’t worked but your hint about changing the encoding afterwards has worked very well!

Bas de Reuver

@peterjones That is indeed the cause of the problem, thanks for the thorough answer👍

It has to do with ANSI<->UTF8 conversion, and the letters with diacritics (à é ü etc.) are incorrectly split into two characters, “Bünder Voll” becomes “BÃ¼nder Voll”.

When the CSVLint plug-in reformats the csv file, I use a StringBuilder to create the new reformatted file content:

StringBuilder datanew = new StringBuilder();

// .. do the reformat

// update text in editor
scintillaGateway.SetText(datanew.ToString());

Right up until the line SetText(datanew.ToString()) the StringBuilder contains the correct “Bünder Voll” etc. values. So I suspect the problem is in scintillaGateway.SetText(?).

It’s good that there is a work-around, but I don’t know how to fix this properly.

PeterJones

@bas-de-reuver ,

I cannot give you the answer, because I don’t know the StringBuilder or scintillaGateway requirements. But my guess is that .ToString() is outputting the data in a different encoding than .SetText() wants coming in, hence the corruption of the encoding. Maybe look into the APIs of both, and see if either come with an option to change the encoding. Or if they don’t have that, then maybe you need some translator in between: scintillaGateway.SetText(translate(datanew.ToString()))

Sorry I cannot be more specific than that.

rdipardo

@bas-de-reuver,

Right up until the line SetText(datanew.ToString()) the StringBuilder contains the correct “Bünder Voll” etc. values.

That’s because the API detects the file encoding for you:

  /// <summary>
  /// Reads the whole document as a text stream, trying to use the right encoding
  /// </summary>
  public static StreamReader StreamAllText()
  {
    var doc = PluginBase.CurrentScintillaGateway;
    var codepage = doc.GetCodePage();
    var encoding = codepage == (int)SciMsg.SC_CP_UTF8 ? Encoding.UTF8 : Encoding.Default;
    return new StreamReader(StreamAllRawText(), encoding);
  }

Problem is, a StringBuilder is just a simple utility with no encoding property that you can set, so the text returned by ToString() will be encoded in the system’s default (usually single-byte) code page.

Creating a StreamWriter with the StreamWriter(Stream, Encoding) overload would be more useful. The second parameter could be set by calling scintillaGateway.GetCodePage() and choosing an appropriate System.Text.Encoding based on the return value (as in the API method shown above). Scintilla doesn’t declare unique constants for every possible encoding; SC_CP_UTF8 really stands for “Unicode,” i.e., any multi-byte encoding.

If want to keep the simplicity of StringBuilder, you could always reduce the reformatted text to bytes, encode each one, then recompose them into a string, like this:

              StringBuilder datanew = new StringBuilder();

              // ... do the reformat

             /// try to match the file encoding of the open buffer
             /// <seealso cref="CsvQuery.PluginInfrastructure.ScintillaStreams.StreamAllText"/>
             Encoding docEncoding =
                 scintillaGateway.GetCodePage() == (int)SciMsg.SC_CP_UTF8
                 ? Encoding.UTF8
                 : Encoding.Default;

             // update text in editor
             var byteBuf = new char[datanew.Length];
             datanew.CopyTo(0, byteBuf, 0, datanew.Length);
             var dataBytes = docEncoding.GetBytes(byteBuf);
             scintillaGateway.SetText(docEncoding.GetString(dataBytes));

Note The fallback choice of System.Text.Encoding.Default is just for illustration. It’s not recommended in practice on .NET Framework. Besides, every character in the ASCII code page fits inside the CLR Char type (which is always UTF-16).

Alan Kilborn

@bas-de-reuver said in [New Plugin] CSV Lint:

That’s why I’ve created a video to show how you can use this plug-in to validate data, reformat datetime values, split column functions

I got around to watching the video. Very nice intro to the plugin!

Lycan Thrope

@bas-de-reuver ,

I had watched your video earlier, but being involved elsewhere with my developing UDL and associated files needing to be done, I didn’t get to really appreciate what it was offering. However, now that the language is mostly done, for now, I started going back to a project of mine that has been “slow-rolling” and started working on it. One of the things that I was trying to do was break down what were fledgling attempts at a quick database that was huge. The data was all needed, I just didn’t take the time to break them into smaller usable entities while I was making a quick app for data entry, viewing, searching, etc.

I needed to clean up, and I was able to separate in dBASE some of the table information, in this case, customers (actually shippers and receivers but am combining their information under just customers) and I needed to clean up and split a field. I could have probably done it in my environment, but decided to take the time to see if I could use your plug-in to do some of the work and simplify the cleanup. It worked beautifully, and although I could accomplish it by not converting it to CSV, it was so much simpler just to convert the data and split and clean it up via your plugin.

I just wanted to thank you for developing this plugin, and making the video, that, although I didn’t understand all of the capabilities you were mentioning about it at the time, I figured it couldn’t hurt to play with it a little, and I’m very happy I did. Thanks for doing the plugin and video. Keep up the good work. :)

Bas de Reuver

@lycan-thrope said in [New Plugin] CSV Lint:

It worked beautifully

Cool, that is also the goal for this plugin; save time by making the inspecting and cleaning of data easier. So thanks 😀 that’s nice to hear you found it useful.

Tanquen

I’m not able to get this plugin to work. I need to make sure it’s not adding any text/data. I just want to make the CSV data easier to read.
My CSV has a large number of columns and different headers every few rows. It defaults to FixedLength but after changing to CSVDelimited it just adds the text “XML” at the top and nothing changes.

Format=FixedLength
ColNameHeader=False
Col1=XML Text Width 9999

Format=CSVDelimited
ColNameHeader=False
Col1=XML Text Width 9999

T Switzer

any update planned to update CSV Lint to work with current version of notepad ++

Lycan Thrope

@t-switzer ,
There already is, but since you haven’t posted which version of NPP you’re using, the assumption is that it is the latest version, and yes, there is an update for it. At present, you’ll need to delete the current version in the plugin folders, or it won’t allow the new NPP to start. Then after you get it started, you can install the newest plugin via the Plugin manager, or go to this site and download it yourself for a self install: CSVList Github page

Bas de Reuver

@Tanquen said in [New Plugin] CSV Lint:

My CSV has a large number of columns and different headers every few rows. It defaults to FixedLength but after changing to CSVDelimited it just adds the text “XML” at the top and nothing changes.

Thanks for mentioning your issue. It sounds like the plug-in can’t recognise this specific data file. I suspect the file includes many < or > characters as well as many , or ; characters or something like that. This can “confuse” the autodetect function so to speak, meaning it can’t determine which is the correct separator character, so it doesn’t interpret the data and columns correctly.

Is it possible to send the data file to my e-mail address (see About dialog)? If it contains privacy sensitive data or is too large, then maybe edit the file and just include a few lines of data to reproduce this issue?

Btw someone metioned a similar issue so in a future update I want to add a where you can (optionally) manually specify the separator character.

In the mean time you can somehow manually construct the meta data, like below.

Format=CSVDelimited
ColNameHeader=False
Col1=Field1 Text Width 50
Col2=Field2 Text Width 50
Col3=Field3 Text Width 50
Col4=Field4 Text Width 50
etc.

Or alternatively, first try to delete the rows (if possible) that are causing trouble, so keep only a few rows with representative data, and then click Refresh from data, and then apply that resulting metadata to the complete file with all the rows.

Bas de Reuver

@T-Switzer said in [New Plugin] CSV Lint:

any update planned to update CSV Lint to work with current version of notepad ++

Like @Lycan-Thrope mentioned, there is a new CSV Lint v0.4.5.2 which you can manually download from the github page. That version will be included automatically in the Plugin Admin in the upcoming Notepad++ v8.4.3.

It looks like the compatibility issues with the new Lexer v5 are solved now 🤞 and I want to wait and see before continuing and adding too many other features to the plug-in.