Hey!
I am using 'Drupal core 7.0 (2011-Jan-05)' and 'Views 7.x-3.0-alpha1 (2011-Jan-06)' in combination with about 60 additional/optional modules.

When I activate 'glossary', there occurs a:

* Notice: Undefined property: views_handler_argument_string::$placeholder_length in views_handler_argument_string->get_formula() (Zeile 147 von /var/www/site.net/sites/all/modules/views/handlers/views_handler_argument_string.inc).
* Notice: Undefined property: views_handler_argument_string::$placeholder_length in views_handler_argument_string->summary_query() (Zeile 127 von /var/www/site.net/sites/all/modules/views/handlers/views_handler_argument_string.inc).

Debug:
'Exception: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near \') AS title_truncated, COUNT(node.nid) AS num_records
FROM 
sec_node node
GROUP B\' at line 1'

in views_plugin_query_default->execute() (Zeile 1314 von /var/www/site.net/sites/all/modules/views/plugins/views_plugin_query_default.inc).

Thanks for fixing and maintinging this great, essential module!

CommentFileSizeAuthor
#8 2010-01-12T02:38_views_glossary-export.txt6.22 KBBlooniverse
#1 1017254-argument_string.patch3.98 KBdawehner

Comments

dawehner’s picture

dawehner
Primary language German
commented
StatusFileSize
new 1017254-argument_string.patch3.98 KB

Does this work for you?

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs review

Update status.

cnolle’s picture

cnolle commented

The above patch fixed the issue for me, thanks.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Fixed

Thanks for testing the patch.

Commited to the 7.x branch.

Blooniverse’s picture

Blooniverse commented
Status: Fixed » Patch (to be ported)

... the most current 'Views' (2011-Jan-11) still shows the error pointed out above. Sorry! Status changed.

merlinofchaos’s picture

merlinofchaos commented
Version: 7.x-3.0-alpha1 » 7.x-3.x-dev
Status: Patch (to be ported) » Active

Wrong status used.

dawehner’s picture

dawehner
Primary language German
commented

It's working perfect for me here.

I have the default glossary of views tested. Both the overview and a listing of a single char worked fine.

Perhaps you have a changed view. If you have one, please provide a reusable export of the view. It would be cool if this export would be without any kind of fieldapi fields or something similar.

As alternative it could be that you missed some files on the latest update.

Blooniverse’s picture

Blooniverse commented
StatusFileSize
new 2010-01-12T02:38_views_glossary-export.txt6.22 KB

Hey @dereine! Thanks for the info.

NO, I don't make use of an adapted 'glossary' view. Nonetheless I have attached the export as a .txt-file below.

BTW: I like your presentations (last time during DrupalCamp in Essen, 2010). I gave the tutorial-lecture 'Drupal-Mobiliser' there.

Blooniverse’s picture

Blooniverse commented

... oh, I've just seen '7.x-3.x-dev (2011-Jan-12)'. Testing 'glossary' again, now! Strange, Drupal doesn't indicate ('/admin/reports/updates') that there is an update available. And Drush?: Neither! Okay, downloading mannually. Feedback in a few minutes.

Blooniverse’s picture

Blooniverse commented

WORKS FINE NOW!

Thank you for 'Views 7.x-3.x-dev (2011-Jan-12)'. Although the 'glossary' view is not important for me, I still want it to work, obviously.

Blooniverse’s picture

Blooniverse commented
Status: Active » Fixed
webankit’s picture

webankit commented

Content taxonomy not working properly ... in gloassary
[{"command":"settings","settings":{"basePath":"\/spic\/","ajaxPageState":{" .....
bla bla etc etc

bojanz’s picture

bojanz commented

@babbar.ankit
That's a separate issue. Try with the latest dev, if it still doesn't work, open a new issue.

tinny’s picture

tinny commented

i get the same error as the op.

AgaPe’s picture

AgaPe commented

life is hard :) with dev version it works but something else then doesnt (default images do not display with views -dev but do with alpha1 version...)

dawehner’s picture

dawehner
Primary language German
commented

The default image seems to be another issue.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

NFL36’s picture

NFL36 commented

How do you install the patch work for #1?

bojanz’s picture

bojanz commented

The patch is a part of the latest -dev version. No need to apply it.