C++ Builder Support

[shaunroselt]

Describe the bug
I installed this into my RAD Studio IDE and it works fine for Delphi and C++. When I use Delphi files, then it shows I am using Delphi and when I use C++ files, then it says I am using C++.

But the issue is with the Editor. It says I am using Delphi for both Delphi and C++. It should say C++ Builder as my Editor when I'm using C++ files.

To Reproduce
Steps to reproduce the behavior:

1. Work on any C++ Builder Project

Expected behavior
I would expect my Editor to be C++ Builder, but I only see Delphi:

[diegomgarcia]

Nice catch, I will add a filter on the file type to change the editor to C++ Builder. However I'm relocating to another country so I expect to get into this issue on next month.

[shaunroselt]

I'm thinking now.

If you add this feature to show C++ Builder, then it probably makes sense to rename this repo to "rad-studio-wakatime" as it's not just Delphi anymore.

[diegomgarcia]

Hey @shaunroselt, I think it make sense. First we just have to check if creating just the filter will fix everything for C++ or if it will raise some new issues. If something like this happens it would be better to separate into two packages.

Lets test it as soon as I have a little more time to get into this issue to work on.

[alanhamlett]

The editor displayed on your dashboard comes from parsing the heartbeat's User-Agent header or plugin json post value. For C++ Builder to be parsed I'll need to first add it to the list of supported editors.

[shaunroselt]

The editor displayed on your dashboard comes from parsing the heartbeat's User-Agent header or plugin json post value. For C++ Builder to be parsed I'll need to first add it to the list of supported editors.

Can we perhaps get it added so long then, please?

It can probably have the same color as Delphi since its logos are pretty much the same in terms of dominant colors.

[alanhamlett]

Added C++ Builder to the list of supported IDEs. The User-Agent header will need to be changed when the plugin detects itself running in C++ builder from:

delphi-wakatime/src/WakaTimeSendHeartbeatThread.pas

Line 107 in 91461c9

UserAgent = 'delphi/' + DelphiVersion + ' delphi-wakatime/' + PluginVersion;

To:

UserAgent = 'delphi/' + DelphiVersion + ' cppbuilder-wakatime/' + PluginVersion;

[diegomgarcia]

Thanks @alanhamlett it's on my todo list now ;)

[diegomgarcia]

Hey @shaunroselt can you test the latest commit from the development branch to see if it works?

[shaunroselt]

Hey @shaunroselt can you test the latest commit from the development branch to see if it works?

Ah. You beat me to it 😁. I wanted to add C++ Builder support this weekend 😂.

Thanks. I'll test the development branch over the weekend and get back to you.

[shaunroselt]

I ended up never testing the development branch.

Is C++ Builder support still planned?

I'm still keen on this feature. It would be nice to have.