February 2006 - openguides-tickets - Urchin mailing lists

[OpenGuides] #72: Node renaming tool
by OpenGuides 15 Dec '06

15 Dec '06

#72: Node renaming tool ---------------------------------------+------------------------------------ Reporter: earle | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: utilities, database, admin | ---------------------------------------+------------------------------------ There should be a tool for admins to rename nodes. As I see it the steps of doing so are: {{{ #!html <ol> <li>Pick node, new node name.</li> <li>Get error if new name exists. Otherwise, confirm rename yes/no.</li> <li>All node data, metadata gets copied with new name.</li> <li>For each node in the metadata shown as linking to the old node, copy the content, replace the old name with new in the content and save it.</li> <li>Replace occurrences of old name with new in link metadata just mentioned.</li> <li>Delete all node data for old node.</li> <li>Display confirmation.</li> </ol> }}} -- Ticket URL: <http://dev.openguides.org/ticket/72> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 5

[OpenGuides] #60: Default node content
by OpenGuides 11 Dec '06

11 Dec '06

#60: Default node content ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- We should prepopulate some choice nodes with some skeletal text to make the initial install less intimidating (especially the home page and those linked in the navbar). -- Ticket URL: <http://dev.openguides.org/ticket/60> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #81: Make "Category" and "Locale" prefixes in autogenerated nodes optional or customisable
by OpenGuides 11 Dec '06

11 Dec '06

#81: Make "Category" and "Locale" prefixes in autogenerated nodes optional or customisable ------------------------+--------------------------------------------------- Reporter: grimoire | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: 0.51 | Severity: normal Keywords: | ------------------------+--------------------------------------------------- I believe that having "Category" and "Locale" prefixed to the name of a node is redundant and counterintuitive for users. A node is a category if it is in category Category, and a locale if it is in category Locale. Also, my Guide is becoming full of redirects from Locale Foo to Foo, and Category Bar to Bar (and occasionally from both Locale Foo and Category Foo to Foo). Several places in the OpenGuides code seem to require that locales and categories have node names prefixed with this information, so it's easier said than done, sadly. -- Ticket URL: <http://dev.openguides.org/ticket/81> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 12

[OpenGuides] #85: Custom user stylesheets
by OpenGuides 11 Dec '06

11 Dec '06

#85: Custom user stylesheets ------------------------+--------------------------------------------------- Reporter: ivorw | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | ------------------------+--------------------------------------------------- I think it would be really useful if people could set their own CSS url in user prefs. This would make developing look & feel so much easier. However, I'm wondering if we want the user CSS in addition to the site one or instead. What do others think? -- Ticket URL: <http://dev.openguides.org/ticket/85> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #48: Edit conflict page erroneously converts lat/lon to os_x, os_y?
by OpenGuides 11 Dec '06

11 Dec '06

#48: Edit conflict page erroneously converts lat/lon to os_x, os_y? -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | -------------------+-------------------------------------------------------- Needs investigation. Reported by crschmidt. -- Ticket URL: <http://dev.openguides.org/ticket/48> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #79: Strip out HTML in node names
by OpenGuides 11 Dec '06

11 Dec '06

#79: Strip out HTML in node names -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: high | Component: openguides Version: svn | Severity: normal Keywords: | -------------------+-------------------------------------------------------- A new spamming ploy is to use <a href ... node names. This results in much easier linkspamming and should be filtered out. -- Ticket URL: <http://dev.openguides.org/ticket/79> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 7

[OpenGuides] #78: deleting versions out of order results in inconsistency
by OpenGuides 11 Dec '06

11 Dec '06

#78: deleting versions out of order results in inconsistency -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: 0.51 | Severity: normal Keywords: | -------------------+-------------------------------------------------------- Just a quick one to remind me to look in more detail. If I delete version 2 out of 3 of a node, the comments end up being inconsistent. Bug in the delete function I guess. -- Ticket URL: <http://dev.openguides.org/ticket/78> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 1

[OpenGuides] #76: Interwiki linking
by OpenGuides 10 Dec '06

10 Dec '06

#76: Interwiki linking ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | ------------------------+--------------------------------------------------- Have a nice way of linking to other openguides wikis: {{{ [[OpenGuides:Oxford:Node Name|Link Text]] }}} This would probably require a cachable http://openguides.org/guide_directory.txt to resolve guide names into URLs. -- Ticket URL: <http://dev.openguides.org/ticket/76> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #82: Search link from Kake's perl.com article to london is broken
by OpenGuides 10 Dec '06

10 Dec '06

#82: Search link from Kake's perl.com article to london is broken ---------------------+------------------------------------------------------ Reporter: grimoire | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: 0.50 | Severity: normal Keywords: | ---------------------+------------------------------------------------------ While reading [http://www.perl.com/pub/a/2003/10/31/openguides.html Kake's perl.com OpenGuides article], I tried clicking the link for "[http://london.openguides.org/supersearch.cgi?lat=51.509186;long=-0.132106;d… find me everything within half a kilometre of Piccadilly Circus Tube station]". At the time of creating this ticket, it doesn't work, presumably because the names of the CGI parameters have changed. This is mildly embarrasing. -- Ticket URL: <http://dev.openguides.org/ticket/82> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #77: Coordinates are off when viewed from Google
by OpenGuides 09 Dec '06

09 Dec '06

#77: Coordinates are off when viewed from Google --------------------+------------------------------------------------------- Reporter: ivorw | Owner: earle Type: defect | Status: new Priority: normal | Component: london.openguides.org Version: svn | Severity: normal Keywords: geodata | --------------------+------------------------------------------------------- Google Maps has highlighted that the Lat/Long coordinates are offset from the maps. see The Millenium Dome, for an example: http://www.nurflax.net/cgi- bin/openguides/wiki.cgi?action=index;format=map&long=0.006849&lat=51.503005&zoom=1 -- Ticket URL: <http://dev.openguides.org/ticket/77> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 5

[OpenGuides] #63: RecentChanges view does not set not_editable tt_var
by OpenGuides 13 Sep '06

13 Sep '06

#63: RecentChanges view does not set not_editable tt_var -------------------------------+-------------------------------------------- Reporter: ChristopherSchmidt | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | -------------------------------+-------------------------------------------- On RecentChanges view, the not_editable variable is not set, so an "edit" link is presented for the node, for which content will never be seen because of special casing in the OG code. -- Ticket URL: <http://dev.openguides.org/ticket/63> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 6

[OpenGuides] #59: Design improvements
by OpenGuides 08 Sep '06

08 Sep '06

#59: Design improvements ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: oxford.openguides.org Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- Get a classic dreaming spires photo up as a banner with some nice title text. I have a suitable photo. Will require hacking CSS/templates too. -- Ticket URL: <http://dev.openguides.org/ticket/59> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #57: Dependencies for tickets
by OpenGuides 08 Sep '06

08 Sep '06

#57: Dependencies for tickets ------------------------+--------------------------------------------------- Reporter: earle | Owner: dom Type: enhancement | Status: new Priority: normal | Component: trac Version: | Severity: normal Keywords: tickets | ------------------------+--------------------------------------------------- Since Trac doesn't seem to have a concept of tickets relying on each other, could we have a custom text field (see TracTicketsCustomFields) on tickets to enter the numbers of other tickets that need to be fixed first? -- Ticket URL: <http://dev.openguides.org/ticket/57> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 3

[OpenGuides] #55: Normal RecentChanges output doesn't accept username=
by OpenGuides 08 Sep '06

08 Sep '06

#55: Normal RecentChanges output doesn't accept username= ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- action=rc;format=rss;username=Foo works. But action=rc;username=Foo doesn't. -- Ticket URL: <http://dev.openguides.org/ticket/55> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 9

[OpenGuides] #49: Remove obsolete metadata item "Map Link"
by OpenGuides 08 Sep '06

08 Sep '06

#49: Remove obsolete metadata item "Map Link" ------------------------+--------------------------------------------------- Reporter: ivorw | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- The "Map Link" field was originally put in as part of the migration from UseModWiki for GrubStreet. This was before the geo coordinates were abstracted out as separate metadata. As it is, "Map Link" usually does not convey any useful information which does not appear elsewhere in the metadata, and is not present in the RDF, hence cannot be mirrored. The exception to this might be sites that have their own custom "how to get there" page, but these are few and far between, and of limited use. Each site can generate its own map links by processing the lat/long, OS_X/OS_Y or postcode, into a URL to an external mapping site. We can do much better than this, as is apparent on [http://boston.openguides.org]. This could easily be turned into a plugin, or probably just as easily be done in template logic. -- Ticket URL: <http://dev.openguides.org/ticket/49> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #43: Show priority in tickets-ordered-by-last-change report
by OpenGuides 08 Sep '06

08 Sep '06

#43: Show priority in tickets-ordered-by-last-change report -------------------+-------------------------------------------------------- Reporter: earle | Owner: dom Type: task | Status: new Priority: normal | Component: trac Version: | Severity: normal Keywords: | -------------------+-------------------------------------------------------- It would be nice if [http://dev.openguides.org/report/9 this report] could have a "priority" column. -- Ticket URL: <http://dev.openguides.org/ticket/43> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 3

[OpenGuides] #38: Link styles are ugly
by OpenGuides 08 Sep '06

08 Sep '06

#38: Link styles are ugly -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | -------------------+-------------------------------------------------------- From http://oxford.openguides.org/wiki/?Comments: Square brackets around links to external pages look ugly. Is there any chance of preventing them from being displayed? -- James * Not that I know of. I actually disagree in any case; it's a useful way of distinguishing between internal and external links, so I'd rather the convention was kept. --Dom * Couldn't that be done within the stylesheet? --James * Stylesheet sounds like the best plan. I'll look into it unless someone else does first. --Kake -- Ticket URL: <http://dev.openguides.org/ticket/38> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #37: No CSS file in distribution
by OpenGuides 08 Sep '06

08 Sep '06

#37: No CSS file in distribution -------------------+-------------------------------------------------------- Reporter: dom | Owner: Type: defect | Status: new Priority: normal | Version: Severity: normal | Keywords: -------------------+-------------------------------------------------------- We really need a decent neat, not too flashy stylesheet to distribute so that the default install isn't so fugly. The specification for the CSS classes and ids is source:trunk/README.CSS. -- Ticket URL: <http://dev.openguides.org/ticket/37> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 5

[OpenGuides] #69: Country-independent date style in Trac
by OpenGuides 01 Jun '06

01 Jun '06

#69: Country-independent date style in Trac ------------------+--------------------------------------------------------- Reporter: earle | Owner: dom Type: task | Status: new Priority: low | Component: trac Version: | Severity: minor Keywords: | ------------------+--------------------------------------------------------- At present dates in Trac are listed as mm/dd/yyyy. Could we possibly get them as yyyy/mm/dd? -- Ticket URL: <http://dev.openguides.org/ticket/69> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #53: Update CGI::Wiki::Formatter::UseMod for compatibility with the latest Text::WikiFormat
by OpenGuides 01 Jun '06

01 Jun '06

#53: Update CGI::Wiki::Formatter::UseMod for compatibility with the latest Text::WikiFormat -------------------+-------------------------------------------------------- Reporter: ivorw | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: 0.51 | Severity: normal Keywords: | -------------------+-------------------------------------------------------- This is currently failing tests and breaking some installs, though workarounds are available. see [http://openguides.org/mail/openguides-dev/2005- November/thread.html#1086] -- Ticket URL: <http://dev.openguides.org/ticket/53> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 1

[OpenGuides] #84: Completely redesign openguides.org
by OpenGuides 24 May '06

24 May '06

#84: Completely redesign openguides.org ------------------------+--------------------------------------------------- Reporter: earle | Owner: earle Type: enhancement | Status: new Priority: normal | Component: website Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- Nuff said. Anybody know any designers? -- Ticket URL: <http://dev.openguides.org/ticket/84> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 1

[OpenGuides] #39: Spell checker
by OpenGuides 25 Apr '06

25 Apr '06

#39: Spell checker ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- From http://oxford.openguides.org/wiki/?Comments: James: My spelling is bad I know. Is it possible to add some sort of spell check? Even if it's just a list of mispelled words so I can go back and re-edit the entry? * Not my itch; patches welcome :) --Kake -- Ticket URL: <http://dev.openguides.org/ticket/39> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #56: Config options for license data
by OpenGuides 20 Apr '06

20 Apr '06

#56: Config options for license data ------------------------+--------------------------------------------------- Reporter: earle | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: 0.51 | Severity: normal Keywords: config | ------------------------+--------------------------------------------------- We should have config options called licence_url and licence_info_url. The former would be used to link to the canonical page about a licence (e.g. something on creativecommons.org); the latter a local page about the licecse (like "Site Licence" on London). -- Ticket URL: <http://dev.openguides.org/ticket/56> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 6

[OpenGuides] #54: RSS redirect breaks username= parameter
by OpenGuides 20 Apr '06

20 Apr '06

#54: RSS redirect breaks username= parameter -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | -------------------+-------------------------------------------------------- action=rss;username=Foo should redirect to action=rc;format=rss;username=Foo but it redirects to action=rc;format=rss. -- Ticket URL: <http://dev.openguides.org/ticket/54> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 3

[OpenGuides] #80: Linking to a node with no name causes a crash
by OpenGuides 20 Apr '06

20 Apr '06

#80: Linking to a node with no name causes a crash -------------------+-------------------------------------------------------- Reporter: ivorw | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | -------------------+-------------------------------------------------------- See http://manchester.openguides.org/?id=Wheel_Of_Manchester&version=1 for an example Putting a wiki link [[|foo]] causes the wikiformatter to b0rk. -- Ticket URL: <http://dev.openguides.org/ticket/80> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 5

[OpenGuides] #42: "About" output
by OpenGuides 08 Apr '06

08 Apr '06

#42: "About" output ------------------------+--------------------------------------------------- Reporter: earle | Owner: earle Type: enhancement | Status: new Priority: low | Component: openguides Version: | Severity: minor Keywords: rdf | ------------------------+--------------------------------------------------- There should be an "action=about" method in wiki.cgi that produces an "About this guide" page, with information about the OpenGuides install (version number, logo, link to website, etc). I figure a link to this could replace the standard OpenGuides link at the bottom of every page. There should also be a link on that page to "action=about;format=rdf" which would produce [http://usefulinc.com/doap/ DOAP] output about the software. -- Ticket URL: <http://dev.openguides.org/ticket/42> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 5

[OpenGuides] #70: "Things nearby"
by OpenGuides 06 Apr '06

06 Apr '06

#70: "Things nearby" -----------------------------------------+---------------------------------- Reporter: earle | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: metadata, geodata, templates | -----------------------------------------+---------------------------------- Every page should have a "things nearby" listing if it has had geodata specified. For an example, see the "other pubs nearby" section of [http://www.beerintheevening.com/pubs/s/61/6160/Sheepwalk/Leytonstone this page at BITE]. -- Ticket URL: <http://dev.openguides.org/ticket/70> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 1

[OpenGuides] #83: Rationalise setup for london.openguides.org
by OpenGuides 05 Apr '06

05 Apr '06

#83: Rationalise setup for london.openguides.org -------------------+-------------------------------------------------------- Reporter: earle | Owner: earle Type: task | Status: new Priority: normal | Component: london.openguides.org Version: | Severity: normal Keywords: | -------------------+-------------------------------------------------------- London.og is a mess internally, featuring a number of custom hacks and old bits of junk lying around, making it extremely tedious to upgrade. Time needs to be taken to sort it out properly. -- Ticket URL: <http://dev.openguides.org/ticket/83> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 4

[OpenGuides] #87: Pass through source for mirror sites
by OpenGuides 05 Mar '06

05 Mar '06

#87: Pass through source for mirror sites ------------------------+--------------------------------------------------- Reporter: ivorw | Owner: ivorw Type: enhancement | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | ------------------------+--------------------------------------------------- A mirror site should really be read only. But, how should an edit request be dealt with? Granted the mirror site could have the templates patched to remove the edit link. But this wouldn't stop spammers. Also, this wouldn't solve the problem of mirrors that also contain some of their own content. The way to detect that a page is mirrored, is that there is a 'source' metadata item, containing the URL of the page on another guide. This should be passed through to the templates, which can do the necessary. This change will have no effect on guides which are not mirrors. -- Ticket URL: <http://dev.openguides.org/ticket/87> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #66: Locales in RDF should use dc:title, not foaf:name
by OpenGuides 05 Mar '06

05 Mar '06

#66: Locales in RDF should use dc:title, not foaf:name -------------------+-------------------------------------------------------- Reporter: earle | Owner: earle Type: defect | Status: new Priority: normal | Component: london.openguides.org Version: svn | Severity: normal Keywords: rdf | -------------------+-------------------------------------------------------- {{{ <foaf:based_near> <wn:Neighborhood> <foaf:name>Streatham</foaf:name> </wn:Neighborhood> </foaf:based_near> }}} foaf:name has a domain of foaf:Person, which is disjoint with wn:Neighbourhood. dc:title would be more appropriate. -- Ticket URL: <http://dev.openguides.org/ticket/66> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 10

[OpenGuides] #75: Google maps dms bug
by OpenGuides 05 Mar '06

05 Mar '06

#75: Google maps dms bug -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | -------------------+-------------------------------------------------------- Google maps support erroneously tries to use dms lat/longs rather than decimal if users select it. -- Ticket URL: <http://dev.openguides.org/ticket/75> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

[OpenGuides] #46: Google maps integration
by OpenGuides 05 Mar '06

05 Mar '06

#46: Google maps integration ------------------------+--------------------------------------------------- Reporter: dom | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: | Severity: normal Keywords: | ------------------------+--------------------------------------------------- Google maps as an index type, on nodes and search results would be cool. Like http://boston.openguides.org/ has, in fact. -- Ticket URL: <http://dev.openguides.org/ticket/46> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 7

[OpenGuides] #68: Malformed user name input not blocked by preferences template
by OpenGuides 05 Mar '06

05 Mar '06

#68: Malformed user name input not blocked by preferences template --------------------------+------------------------------------------------- Reporter: earle | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: templates cgi | --------------------------+------------------------------------------------- If you put something like {{{ <a href="http://example.com/">Foo</a> }}} into the preferences page as your username, the edit form gets broken. (See attachment.) The username should be unescaped or some such before being set in the preferences cookie. -- Ticket URL: <http://dev.openguides.org/ticket/68> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 1

[OpenGuides] #52: "Locales" to be renamed "Areas"
by OpenGuides 05 Mar '06

05 Mar '06

#52: "Locales" to be renamed "Areas" --------------------------------+------------------------------------------- Reporter: earle | Owner: dom Type: enhancement | Status: new Priority: normal | Component: openguides Version: 0.51 | Severity: normal Keywords: templates internals | --------------------------------+------------------------------------------- As discussed on IRC on Nov 28, the name "Locales" should be replaced - at least in the user view, internally is not that important* - with "Areas". * Or is it? Am I being lazy here? Maybe changing the name internally is a good idea to avoid future confusion, bit rot, etc. -- Ticket URL: <http://dev.openguides.org/ticket/52> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 3

[OpenGuides] #86: redirects under mod_perl
by OpenGuides 21 Feb '06

21 Feb '06

#86: redirects under mod_perl -------------------+-------------------------------------------------------- Reporter: dom | Owner: dom Type: defect | Status: new Priority: normal | Component: openguides Version: svn | Severity: normal Keywords: | -------------------+-------------------------------------------------------- From crschmidt: Currently, when running OpenGuides under mod_perl, redirects do not work, because the Apache $r object associated with the CGI object do not exist. (You get an error message: Can't call method "send_cgi_header" on an undefined value at (eval 64) line 62.) This happens both after an edit, and when following redirects, and possibly in other cases as well. After some investigation last night, I tracked the source of this down: currently, OpenGuides redirects use a class method of CGI: {{{ CGI->redirect($redirectparams); }}} However, this does not work. It seems that class methods in this case never initialize the Apache request variable that is needed in order to send_cgi_header. The solution for this, is to replace CGI->redirect calls with: {{{ my $q = new CGI; $q->redirect($redirectparams); }}} I have too little knowledge to be able to speak to whether this is a fault within CGI or a fault within OpenGuides usage of it. However, this change has allowed me to successfully use mod_perl on the Open Guide to Boston in testing, and this fix or a similar fix should probably be integrated into OpenGuides in either case, since upgrading the system CGI.pm is a difficult task. The current debian packages for stable include mod_perl 1.999.021, which is not compatible with CGI.pm 3.16, and upgrading mod_perl requires a full removal due to an API rename (as described at http://perl.apache.org/docs/2.0/rename.html). The code that needs to be changed is in OpenGuides.pm, in two functions: {{{ sub redirect_to_node sub find_within_distance }}} A patch which makes these changes is attached. -- Ticket URL: <http://dev.openguides.org/ticket/86> OpenGuides <http://openguides.org/> The OpenGuides city guides project

1 2

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

openguides-tickets February 2006