Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

Setting component=* for /newticket does not work #4754

Open
openstreetmap-trac opened this issue Jul 23, 2021 · 9 comments
Open

Setting component=* for /newticket does not work #4754

openstreetmap-trac opened this issue Jul 23, 2021 · 9 comments

Comments

@openstreetmap-trac
Copy link

Reporter: skyper
[Submitted to the original trac issue database at 4.37pm, Wednesday, 23rd January 2013]

All the new ticket links on the front page do not really work as the component is not set although the syntax seems to be OK.

According to wiki:TracTicket this should work:[[BR]]
/newticket?summary=Compile Error&version=1.0&component=wiki

I miss a component trac or is this admin ?

Cheers

@openstreetmap-trac
Copy link
Author

Author: skyper
[Added to the original trac issue at 4.52pm, Wednesday, 23rd January 2013]

Replying to [ticket:4754 skyper]:

Why can I not edit ticket description of which I am the reporter ?

According to wiki:TracTicket this should work:[[BR]]

Sorry, wiki:TracTickets#PresetValuesforNewTickets

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 5.25pm, Wednesday, 23rd January 2013]

I think this is a side effect of the javascript hack we use to ensure that no component is selected by default. I did try and fix it but have so far been unable to do so.

@openstreetmap-trac
Copy link
Author

Author: ppawel[at]fastmail.fm
[Added to the original trac issue at 9.06am, Thursday, 24th January 2013]

I'm 99% positive that it has been fixed some weeks ago in #4714 after I reported it.

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 9.14am, Thursday, 24th January 2013]

Well the file hasn't changed since 10th December and there is no sign of anything in the code in there that would have fixed this.

The basic problem is that I can't find a way in javascript to reliably tell the difference between an that has been selected by default because it is the first in the list and one which has an explicit "selected" attribute.

@openstreetmap-trac
Copy link
Author

Author: ppawel[at]fastmail.fm
[Added to the original trac issue at 9.22am, Thursday, 24th January 2013]

Perhaps you could check if there's the "component" parameter in the query string and only if there's not, you add the empty option?

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 9.23am, Thursday, 24th January 2013]

Well I was trying to avoid having to parse the URL because there is no easy way to do that in Javascript!

@openstreetmap-trac
Copy link
Author

Author: TomH
[Added to the original trac issue at 9.28am, Thursday, 24th January 2013]

Right. I think I managed to fix this - well it works in Firefox anyway...

@openstreetmap-trac
Copy link
Author

Author: skyper
[Added to the original trac issue at 1.03pm, Thursday, 24th January 2013]

Does not work for me on Debian 7.0.

Tested with:

  • Mozilla Iceweasel 10.0.12
  • Chromium 22.0.1229.94 Debian 7.0
  • Epiphany Web 3.4.2

@openstreetmap-trac
Copy link
Author

Author: ppawel[at]fastmail.fm
[Added to the original trac issue at 9.58am, Monday, 11th February 2013]

Confirmed not working in Chromium...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant