Comments on: P4A 3.0.1 released

By: Fabrizio Balliano

Fabrizio Balliano — Wed, 09 Jul 2008 09:14:05 +0000

@Eddie:added with multi-pk support :-) thank you for suggestion!

By: Fabrizio Balliano

Fabrizio Balliano — Wed, 09 Jul 2008 07:21:09 +0000

@Eddie: I’ll take a look at your second suggestion, for the first please post more info on forums. thank you!

By: Eddie

Eddie — Tue, 08 Jul 2008 23:59:47 +0000

I recommend the modification of the line 742 of db_source to incorporate something like:

$pks = $this->getPk();
$pkvalue = $p4a_db_table->insert($fields_values);

if (is_string($pks) && !is_array($pkvalue)) {
$this->fields->$pks->setValue($pkvalue);
}

By: Eddie

Eddie — Tue, 08 Jul 2008 23:48:27 +0000

Hi Fabrizio, I use an auto_increment field for my tables, why after insert a record via saveRow() method, the pointer goes to te first row..? I’ve traced the saveRow() and the value of my pk field is actually gathered, but never is setted to the P4A_DB_Field to allow save the multivalue fields and locate the new row.

By: Fabrizio Balliano

Fabrizio Balliano — Fri, 04 Jul 2008 08:52:25 +0000

yeah nice suggestion, please post a feature request so we’re not going to forget this :)

By: blas

blas — Fri, 04 Jul 2008 08:42:44 +0000

Fabrizio,

I want to make a comment (or maybe suggestion) about setSource…

I see that p4a uses the info zend_db_table method in order to know the fields of a table and creates it in the mask.

I’m thinking that, could be, it will be a useful enhacement, in a future version, using the metadata information provided by this call to set another field property as length (and maybe width ) or some validator (as nullable, type, etc.) by default.

I don’t know if this culd be possible or I forgot something …

TIA,

blas

By: Fabrizio Balliano

Fabrizio Balliano — Wed, 02 Jul 2008 08:22:04 +0000

@Eddie: thank you, I’ll give that a look asap!

By: Eddie

Eddie — Tue, 01 Jul 2008 20:40:33 +0000

Fabrizio, using the exportToCSV method of P4A_DB_Source, I noted that the columns are disordered ’cause the Joins; I modified the code to export the data in the order that the $fields_names are setted:

public function getAsCSV($separator = ‘,’, $fields_names = null)
{
if ($fields_names === true or is_array($fields_names)) {
$insert_header = true;
} else {
$insert_header = false;
}

if ($fields_names === null or $fields_names === false or $fields_names === true) {
$fields_names = array();
while ($field = $this->fields->nextItem()) {
$name = $field->getName();
$fields_names[$name] = $name;
}
}

$csv = “”;
$rows = $this->getAll();

if ($insert_header) {
array_unshift($rows, $fields_names);
}

foreach ($rows as $row) {
$strrow = “”;
foreach ($fields_names as $field=>$label) {
if (array_key_exists($field, $row)) {
$col = str_replace(“\n”,”",$row[$field]);
$col = str_replace(“\r”,”",$col);
$strrow .= ‘”‘ . str_replace(‘”‘,’”"‘,$col . “\”{$separator}”);
}
}
$csv .= substr($strrow,0,-1) . “\n”;
}
return $csv;
}