Flyspray - The bug killer!

  • Status Closed
  • Percent Complete
    100%
  • Task Type TODO
  • Category Backend/Core
  • Assigned To
    Psychokiller1888
  • Operating System All
  • Severity Low
  • Priority Very Low
  • Reported Version 1.0 devel (github master)
  • Due in Version 1.0
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Flyspray - The bug killer!
Opened by peterdd - 11.03.2015
Last edited by peterdd - 11.02.2016

FS#1963 - Review/Test current source for 1.0 with .htaccess Rewrite Rule feature

.htaccess enables the mapping of nicer looking URLs to the default get parameter requests.

After all our changes for FS 1.0 we should review if it works well together when enabling .htaccess for the Rewrite Rules.

Maybe we have to update Rewrite Rules if there are added features that are good candidates for nicer urls.

The task depends upon
ID Project Summary Priority Severity Assigned To Progress
1776 Flyspray - The bug killer!  FS#1776 - Arrow showing the last visited task broken when using 'address_r  Very Low Low
100%
The task blocks this from closing
ID Project Summary Priority Severity Assigned To Progress
1973 Flyspray - The bug killer! FS#1973 - url rewrite Very Low Low
80%
Closed by  peterdd
11.02.2016 04:06
Reason for closing:  Implemented in devel
Additional comments about closing:  

I made some commits on github and think it is good enough now for FS1.0

Project Manager
Psychokiller1888 commented on 14.03.2015 14:17

Well, seems to work after:

https://github.com/Flyspray/flyspray/commit/f23f5a49a03a27be7c7e8c32e07d1a4425704eb4

At least main functions are ok. Deeper we shall go for 1.1

I have planned a refactoring for 1.1 ⇒ FS#1973

Project Manager
peterdd commented on 04.09.2015 23:05

Special case

A web hosting environment where mod_rewrite is available, but also mod_negotiation enabled and you cannot switch mod_negotiation off in .htaccess due not having the right to execute 

Options -MultiViews

in your .htaccess

See current htaccess.dist for my comments.

That leads to problems with all "index/" urls. vs index.php .

So the easiest solution/workaround would be to replace 'index' by another default string.
'tasklist' is still existent in code for the task list and for the others just use '/'

And 'index' is not full correct anymore because the 'index' page type is configurable (toplevel, roadmap or task list).

The many places in code with CreateURL('index',...) may probably stay unchanged if it is handled right in files /.htaccess and /includes/class.tpl.php function CreateURL()

Project Manager
peterdd commented on 19.09.2015 05:02

One of the small flaws yet is that some icons of the Lightbox for viewing images cannot load some icons, because they are related loaded to a virtual url path like

/task/123/

Adding a base href-tag would solve this, but there isn't research done yet for possible disadvantages.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing