<br><br><div class="gmail_quote">On Wed, Oct 5, 2011 at 12:22 PM, Daniel Kastl <span dir="ltr"><<a href="mailto:daniel@georepublic.de">daniel@georepublic.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br><br><div class="gmail_quote"><div class="im">On Wed, Oct 5, 2011 at 12:07 PM, Stephen Woodbridge <span dir="ltr"><<a href="mailto:woodbri@swoodbridge.com" target="_blank">woodbri@swoodbridge.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
YuLongzhen,<br>
<br>
Thank you for your report. I think your analysis and solution are correct.<br>
<br>
Daniel or Anton,<br>
<br>
quote_ident() should only be used when quoting a column or table when it is actually being used as a column or table object in the SQL and not when it is being used as a string. But you guys know that and I assume this is just a brain slip when editing the script. Anyway, please update git when you have a chance.<br>
<br>
Thanks,<br>
-Steve</blockquote><div><br></div><div><br></div></div><div>Hi YuLongzhen and Steve,</div><div><br></div><div>As far as I remember assign_vertex_id has almost not been modified since it was part of pgDijkstra many years ago.</div>
<div>So thank you for looking into this, YuLongzhen! Nobody has ever notified this (and reported).</div><div><br></div><div>YuLongzhen, if you want to clone the Github repository, make the modifications and submit a "pull request", then the credits for the patch will be yours. </div>
<div><br></div><font color="#888888"><div>Daniel</div></font></div></blockquote><div><br></div><div></div></div><div><br></div><div>Or <font face="'courier new', monospace">quote_literal()</font> might be even better:</div>
<div><a href="http://www.postgresql.org/docs/8.4/static/plpgsql-statements.html#PLPGSQL-QUOTE-LITERAL-EXAMPLE">http://www.postgresql.org/docs/8.4/static/plpgsql-statements.html#PLPGSQL-QUOTE-LITERAL-EXAMPLE</a></div><br>
<div>
<font face="'courier new', monospace">...</font></div><div><font face="'courier new', monospace">FOR _r IN EXECUTE 'SELECT srid FROM geometry_columns WHERE f_table_name='|| quote_literal(geom_table)||';' LOOP</font></div>
<div><font face="'courier new', monospace"> srid := _r.srid;</font></div><div><font face="'courier new', monospace"> END LOOP;</font></div><div><font face="'courier new', monospace">...</font></div>
<div><br></div><div>Or am I wrong?</div><div>I have never really taken note much at these string functions, though there seem to be many useful ones:</div><div><a href="http://www.postgresql.org/docs/8.4/static/functions-string.html">http://www.postgresql.org/docs/8.4/static/functions-string.html</a></div>
<div><br></div><div>Daniel </div><div><br></div><div><br></div>-- <br><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Georepublic UG & Georepublic Japan<br>eMail: <a href="mailto:daniel.kastl@georepublic.de" style="color:rgb(66, 99, 171)" target="_blank">daniel.kastl@georepublic.de</a><br>
Web: <a href="http://georepublic.de/" style="color:rgb(66, 99, 171)" target="_blank">http://georepublic.de</a></span><br>