Giter Site home page Giter Site logo

Comments (13)

narfbg avatar narfbg commented on April 27, 2024

This really depends on wether #388 is considered a bug or not.

from codeigniter.

narfbg avatar narfbg commented on April 27, 2024

... or not - ignore my last comment.

from codeigniter.

iamjochem avatar iamjochem commented on April 27, 2024

+1 for scrapping the "conversion programatic characters" - it is pointless, has the potential to waste a lot of time 'debugging' and it is incorrect. the code essentially performs a limited "html entitizing" ... html entitizing is something you do to output not input!

from codeigniter.

jim-parry avatar jim-parry commented on April 27, 2024

Hasn't this been address by #388? Can this be closed?

from codeigniter.

narfbg avatar narfbg commented on April 27, 2024

#388 has nothing to do with this ... other issues related to this one have been fixed, but the suggestion here is to remove this filter altogether.

from codeigniter.

Razican avatar Razican commented on April 27, 2024

I think enough people are having issues with it and that it does not have a clear advantage, so in my opinion it should be removed.

from codeigniter.

jim-parry avatar jim-parry commented on April 27, 2024

I agree that the substitution should be removed. RFC 3986 says that the dollar sign and parentheses are safe characters and do not need encoding. They are also flagged as "reserved" characters, which can be encoded and interpreted by an application, but that appears to be subsequent to any use as a URI.

from codeigniter.

ostrolucky avatar ostrolucky commented on April 27, 2024

I'm voting for removing it.

from codeigniter.

ivantcholakov avatar ivantcholakov commented on April 27, 2024

+1 for "conversion programatic characters" removal. The three presented justifications are good enough.


Edit: Justification 4: Such characters within a segment may be needed as a result of sloppy slug generation. If you use url_title() for this purpose, the segment would be clean, and then the "programatic characters" simply may not be enabled using the setting $config['permitted_uri_chars'].

url_title() may be reworked to transliterate from non-Latin languages, but this is another story.

from codeigniter.

Razican avatar Razican commented on April 27, 2024

I vote for it too.

from codeigniter.

narfbg avatar narfbg commented on April 27, 2024

Well, the public opinion seems to be unanimous.

@pfote @benedmunds @druu @lonnieezell Any objections?

from codeigniter.

benedmunds avatar benedmunds commented on April 27, 2024

No objection.

from codeigniter.

lonnieezell avatar lonnieezell commented on April 27, 2024

I can't think of a reason it's really needed, but haven't scoured the code about this either.

No objection.

from codeigniter.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.