Support Bug after installing add-on

[john917]

Can you help please? I purchased and doesn't work Error posted here [Xen-Soluce] Disallow Links [Paid]

[LIST]
[*]ErrorException: Fatal Error: Access level to SV\UserEssentials\Service\Content\SimplifiedPreviewTrait::_validate() must be public (as in class XenSoluce\DisallowLinks\XF\Service\Thread\Creator)
[*]src/addons/SV/UserEssentials/XF/Service/Thread/Creator.php:61
[*]Generated by: test
[*]Apr 15, 2024 at 11:16 PM
[/LIST]
[HEADING=2]Stack trace[/HEADING]
#0 [internal function]: XF::handleFatalError()
#1 {main}
[HEADING=2]Request state[/HEADING]
array(4) {
  ["url"] => string(46) "/snip/post-thread"
  ["referrer"] => string(58) "/snip/custom-og"
  ["_GET"] => array(0) {
  }
  ["_POST"] => array(11) {
    ["_xfToken"] => string(8) "********"
    ["title"] => string(4) "test"
    ["discussion_type"] => string(10) "suggestion"
    ["message_html"] => string(12) "<p>test2</p>"
    ["attachment_hash"] => string(32) "snip"
    ["attachment_hash_combined"] => string(83) "{"type":"post","context":{"node_id":192},"hash":"snip"}"
    ["watch_thread"] => string(1) "1"
    ["_xfSet"] => array(1) {
      ["watch_thread"] => string(1) "1"
    }
    ["_xfRequestUri"] => string(34) "/snip/custom-og"
    ["_xfWithData"] => string(1) "1"
    ["_xfResponseType"] => string(4) "json"
  }
}

 

[CRUEL-MODZ]

Hello,

I'll make you an update to correct this tonight!

Regards, CRUEL-MODZ

 

[john917]

Hello,

I'll make you an update to correct this tonight!

Regards, CRUEL-MODZ

Hi I tried the new version but for some reason the functionality just doesn't work. I have the permissions and also analyzed permissions for a particular user and it should be preventing links but it doesn't. No error this time.

 

[CRUEL-MODZ]

Hello,

Did you activate it in the options?


Regards, CRUEL-MODZ

 

[john917]

Ok, it works now! But ironically it lets users link without the https:// or http://
Is there a way we can have it detect any www. or .com without the http?

 

[john917]

Ok, it works now! But ironically it lets users link without the https:// or http://
Is there a way we can have it detect any www. or .com without the http?

CRUEL-MODZ Seems like we just need to revamp the regex right? $pattern = "#(?P<method>https?://|www)(?P<url>.([a-zA-Z0-9\-\/.]+))#";

Can we implement this one?
(https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|www\.[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9]+\.[^\s]{2,}|www\.[a-zA-Z0-9]+\.[^\s]{2,})

RegExr: Learn, Build, & Test RegEx

RegExr is an online tool to learn, build, & test Regular Expressions (RegEx / RegExp).

regexr.com

 

[CRUEL-MODZ]

I can see if there are the 3 www, but it can generate false positives.

I wouldn't be able to detect this type of thing on xen-soluce.com because there are lots of extensions and if I try to detect what comes after a dot, it's always going to give an error, for example: hello guy.come one, it's going to tell me there's an error when there's not.

And links with the www are becoming less and less common.

 

[john917]

I can see if there are the 3 www, but it can generate false positives.

I wouldn't be able to detect this type of thing on xen-soluce.com because there are lots of extensions and if I try to detect what comes after a dot, it's always going to give an error, for example: hello guy.come one, it's going to tell me there's an error when there's not.

And links with the www are becoming less and less common.

Wait, are you using this formula? RegExr: Learn, Build, & Test RegEx

Here's another one regex101: build, test, and debug regex

Can we make it so we can have the option to use this formula? I don't see why anyone would type guy.come unless dots are allowed in xenforo usernames, which shouldn't be in the first place.

This is good since people try to always bypass things by typing things like google.comcom

 

[CRUEL-MODZ]

Wait, are you using this formula? RegExr: Learn, Build, & Test RegEx

Here's another one regex101: build, test, and debug regex

Can we make it so we can have the option to use this formula? I don't see why anyone would type guy.come unless dots are allowed in xenforo usernames, which shouldn't be in the first place.

This is good since people try to always bypass things by typing things like google.comco

No, because your regex will capture them all, so even Guy.come one is no good.

By the way, xen-soluce.com is not a link and xenforo doesn't detect it as a link. www.xen-soluce.com XF detects it as a link

Because we're on a free editor and we can write anything we want, putting two words together with a dot could give false positives and confuse the user.

I don't think it's a good thing to restrict so much. Especially since xen-soluce.com isn't a link and you could bypass it like this: xen-soluce. com, so it's all an effort for nothing.
. The best thing is to limit it to https/http and www