Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should short.php be removed with WBCE 1.2.0? #234

Closed
cwsoft opened this issue Mar 14, 2017 · 3 comments
Closed

Should short.php be removed with WBCE 1.2.0? #234

cwsoft opened this issue Mar 14, 2017 · 3 comments

Comments

@cwsoft
Copy link
Contributor

cwsoft commented Mar 14, 2017

I would vote for removing short.php and the htaccess.txt file with WBCE 1.2.0 due to the following reasons:

  • it doesn´t work in subfolders or in subdomains (e.g. VirtualHosts) out of the box
  • needs to be adapted in case media/, admin/, pages/ folder is renamed
  • may interfeer with other .htaccess settings (e.g. error document, rewrite rules, PHP handlers etc.)
  • requires knowledge of Apache .htaccess in many cases (target WBCE audience won´t have)
  • simple error in the .htaccess file may result in a Error 500 page easily
  • too many possible options not included (e.g. with or without www, with or without trailing / etc.)
  • it only works on Apache servers, but not on Windows IIS

So I would remove short.php and maybe the [[Shorturl]] Droplet as well. If needed, one can install it from the Dev4Me website.

Any comments, votes on that??

@cwsoft cwsoft added this to the 1.2.0 milestone Mar 14, 2017
@ghost
Copy link

ghost commented Mar 14, 2017

+1 for removal

@instantflorian
Copy link
Contributor

+1 from me too (although it's a pity, but better no solution than this)

@NorHei
Copy link
Contributor

NorHei commented Apr 7, 2017

+1 For me too

... We dont need the Droplet any more , its a filter you can turn on off.
My personal Plan was keeping it in 1.2.x And move this to an Installable Module on 1.3.x
Removing it from Core in 2.0.

As i simply don't wanted to break existing pages that only upgrade from 1.1

@cwsoft
Quite a lot of those Problems occure whith any URL rewriting CMS ;-)

@cwsoft cwsoft closed this as completed Apr 9, 2017
@cwsoft cwsoft modified the milestones: 1.2.0, 2.0.0-dev May 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants