TYPO3 Core

What is wrong with your patch? 140 KB?

I builded a crawler configuration for news records where each category has an own detail page.

my config:
&L=[0|1]&tx_news_pi1[news]=[_TABLE:tx_news_domain_model_news;_PID:9;_FIELD:tx_news_domain_model_news.uid;_ENABLELANG:1;_ADDTABLE:,tx_news_domain_model_news_category_mm,tx_news_domain_model_category;_WHERE: AND tx_news_domain_model_news.uid = tx_news_domain_model_news_category_mm.uid_local AND tx_news_domain_model_news_category_mm.uid_foreign = tx_news_domain_model_category.uid AND tx_news_domain_model_category.single_pid = 20]

in this case the crawler build an query with quoteStr. As example here for the pid of the news records.

$GLOBALS['TYPO3_DB']->quoteStr('pid', 'tx_news_domain_model_news');

with the current code an query will generate which fails because pid has no relation to a table. example: (have a look to the pid field)

SELECT tx_news_domain_model_news.uid FROM tx_news_domain_model_news,tx_news_domain_model_news_category_mm,tx_news_domain_model_category WHERE tx_news_domain_model_news.uid = tx_news_domain_model_news_category_mm.uid_local AND tx_news_domain_model_news_category_mm.uid_foreign = tx_news_domain_model_category.uid AND tx_news_domain_model_category.single_pid = 20 and pid = 9

when the table would be use there would not be a problem. or maybe the crawler should use another method?

This seems to be a bug of the crawler. I'm citing the description of the quoteStr() function:

Unless you have to apply the single quotes yourself you should rather use ->fullQuoteStr()!

@return string Output string; Quotes (" / ') and \ will be backslashed

It is intended to be used for values not for columns. So for columns the user has to take care on its own to have a unique naming.

Typical usage of fullQuoteStr() is:

$GLOBALS['TYPO3_DB']->exec_SELECTquery(
  '*',
  'tx_ext_table',
  'myname = ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('My\'Name/with special chars in "it"', 'tx_ext_table')
);

Thanks for your interest to this problem

I will create a patch for the crawler extension.