Автоматическое создание РНР кода для отображения результата выборки из базы.

[antonio]

Автоматическое создание РНР кода для отображения результата выборки из базы.

Вот собственно сабж.

http://anton.concord.ru/pagination/visualpagination.php

интересуют любые мнения.

 

[Crazy]

1. Идем в [ SQL ].
2. Вводим строку: select " from " as c from Contracts
3. Запрашиваем генерацию кода.

 

[antonio]

Кол-во записей - это не баг, а фича МуСкуля попробуй у ся на любой таблице это провернуть

По поводу кода - забыл addslashes сделать для sql теперь пофиксено, сенькс

 

[Crazy]

Какой баг?

 

[Кром]

Баг в dbclose.php
Открывается mysql

<?
@mssql_close($db);
?>

А вообще кода слишком много.

 

[HEm]

http://anton.concord.ru/pagination/visualpagination.php?CategoryId=7
Warning: array_keys(): The first argument should be an array in c:\inetpub\wwwroot\pagination\form_columns.inc on line 30

 

[antonio]

сенькс, поправлено!

 

[Crazy]

Тест номер раз:

1. Идем в раздел [ SQL ].
2. Вводим вот такой оператор:

select *
from
Contracts

3. Во всех расположенных справа закладках наблюдаем:

You must use SELECT operator only into SQL query!

Тест номер два:

1. Идем все в тот же [ SQL ].
2. Вводим оператор:

select "$foo" from Contracts

3. Скачиваем код и запускаем:

PHP Notice: Undefined variable: foo

 

[Crazy]

Еще тест:

Вот так -- работает:

select distinct ContractNumber from Contracts

И вот так -- тоже работает:

select SUBSTRING(ContractNumber,1,10) c from Contracts

А вот так -- уже дохнет:

select distinct SUBSTRING(ContractNumber,1,10) c from Contracts

 

[antonio]

fixed

 

[Crazy]

Проверяем запрос:

select distinct SUBSTRING(ContractNumber,1,10) c from Contracts

В разделе [ Pagination ] видим:

Records:__[ 1-20 ]___from 10

Видны 10 строк. Меняем запрос:

select distinct SUBSTRING(ContractNumber,1,8) c from Contracts

Меняется и ответ:

Records:__[ 1-20 ]___from 1

Видна одна строка.

Это "1-20" -- так и должно быть?

 

[Crazy]

Пробуем запустить скаченную версию. PHP 4.3.4:

PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in C:\_\php\pagination.php on line 225

PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in C:\_\php\pagination.php on line 245

PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in C:\_\php\pagination.php on line 245

PHP Warning: Call-time pass-by-reference has been deprecated - argument passed by value; If you would like to pass it by reference, modify the declaration of [runtime function name](). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. However, future versions may not support this any longer. in C:\_\php\pagination.php on line 265

PHP Notice: Undefined variable: error in C:\_\php\pagination.php on line 132

 

[antonio]

fixed

 

[antonio]

Видна одна строка.

Это "1-20" -- так и должно быть?

Да, так и должно быть.

 

[Crazy]

1. Заказал "select * from Contracts".
2. Скачал исходники.

Ответ:

Sql syntax error.

Ошибка, полагаю, вот здесь:

if (!$Fields=$rs->MoveNext()) {
  $this->error="Sql syntax error.";
  $this->content.= $rs->errstr;
  return;
}

Hint: таблицы бывают пустыми. И, кстати, не слишком ли жестоко посылать перед этим в базу запрос на извлечение всех записей?

 

[Crazy]

Вопрос: каково назначение атрибута sql1 класса Pagination, вокруг вычисления которого устроено столько танцев с бубном?

 

[antonio]

Не обессудь класс писал даааавно, может и хотел для че-то заюзать
А это горе ща поправим.

 

[antonio]

пофиксил.

 

[antonio]

по поводу полей. А как я тебе их достану тадыть без посылки запроса в базу? Парсер SQL наворачивать не хоца. Можно правда limit 1 в конец добавить, но все равно запрос слать придется.

 

[Crazy]

Автор оригинала: antonio
Не обессудь класс писал даааавно, может и хотел для че-то заюзать

Есть мнение, что это остатки кода, который строил запрос с "select count....". IMHO, без этого кода все остальное особого смысла не имеет. Попробуй на примере:

select * from Contracts a, Contracts b, Contracts c