Search no longer supports values containing slashes ‘/’

  • Resolved Garrett Hyder

    (@garrett-eclipse)


    7 years ago

    Hello,

    It’s been 5 versions now and I’ve had to continuously remove the slash criteria from class_functions.php to rectify search when some values contain slashes;
    || substr_count( $criteria, '/' )

    I raised this over a year ago on an issue that’s been marked resolved and closed to comments prematurely. It was indicated at the time the update would be in place in the next version but has yet to materialize.

    Please reopen this and address – https://www.ads-software.com/support/topic/plugin-changes-broke-search-where-values-contain-slash/

    Thank you

    The page I need help with: [log in to see the link]

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Contributor MariaKravchenko

    (@mariakravchenko)

    Hello.

    We really fixed that issue. You could check the fix here – https://github.com/wp-property/wp-property/issues/494

    However for technical reasons we not able to do release from that branch for now.
    We apologize for the inconvenience.

    Regards.

    Thread Starter Garrett Hyder

    (@garrett-eclipse)

    Thanks @mariakravchenko, greatly appreciate the Git issue it provides the bigger picture and I see it didn’t just drift into the ether.

    I can understand the technical there had just gotten silence on my ticket so wasn’t sure if anything was happening. I’ve now subscribed to the Git ticket so will resolve here and follow there.

    Cheers

    Thread Starter Garrett Hyder

    (@garrett-eclipse)

    @mariakravchenko I did just notice the Git issue is closed. Is there one for the technical issues surrounding the branch that I can follow to get notified when that’s been addressed?
    Thanks

    Plugin Contributor MariaKravchenko

    (@mariakravchenko)

    I think I will just put some comment on that issue when we will move the fix to the branch where from we do releases. So you will see that comment.

    Thread Starter Garrett Hyder

    (@garrett-eclipse)

    Thank you

    Thread Starter Garrett Hyder

    (@garrett-eclipse)

    FYI – I noticed this has now been resolved with the latest update. Thank you

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Search no longer supports values containing slashes ‘/’’ is closed to new replies.