#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
#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
#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
#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
#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