Tested, works as expected.

For __DEPLOY_VERSION__ we have to adjust the build tool.

Btw. Is the issue tracker still working with weblinks? Looks like i can't test this one with it..

the weblink issue tracker is not updated the max(id) is 279 < 288

tested successfully

Btw. Is the issue tracker still working with weblinks?

All of the webhooks for this repo are failing to send data, probably means ModSec is blocking the payloads.

Nevermind, the issue was actually in the webhook configuration on this repo. I've fixed it for you.

I tested this patch successfully

1. I created a weblink.
2. I created a menu item and clicked the weblink one time and I saw one hit.
3. I created an admin module of the type statistic. I see no entry for this in the module because the core patch is merged.
4. After that I copied the system plugin from this patch into the system plugin folder of my installation, discovered it and activated it.
5. After that I saw an entry for my weblink in the statistic module

Only one little point. My code sniffer told me that he expected 2 spaces after the longest type in line 32 oft the class PlgSystemWeblinks.

After "Apply Patch" in Patchtester got Error: The file marked for modification does not exist: pkg_weblinks.xml

@franz-wohlkoenig You can't use patch tester here. The package needs to be rebuild with the patch, see README.md

@yvesh Sorry, had changed in Patchtester the GitHub Repository from "Joomla cms" to "Joomla weblinks package" and thought this works.

Can't we use patch tester only for this patch or can' t we use it at all?
It is already explained in the read.me file :)

Maybe my English isn't good enough so i understand read.me i can test weblinks-repo by patchtester.

Can't we use patch tester only for this patch or can' t we use it at all?

You should be able to test weblinks with patch tester, but the weblinks package would need to be installed first.

Oh, wait, I see the problem now. Why is there a package manifest at both src/pkg_weblinks.xml and src/administrator/manifests/packages/pkg_weblinks.xml?

I tried the patch tester for com_weblinks too this evening.
Patches that contain existing files show the error. But a patch with contains only a new file (#294) shows no error message. It shows a success message but it did not copy the file.

@chrisdavenport can you please sync to master here please?

Closing as this PR is not updated since more than one month. I have opend an new PR here: #333 please test.