2025-06-18 Fred Gleason <fredg@paravelsystems.com>

* Fixed a regression in the 'Editing Event' in rdlogmanager(1) that
	caused double SQL lookups.

Signed-off-by: Fred Gleason <fredg@paravelsystems.com>
This commit is contained in:
Fred Gleason 2025-06-18 16:30:16 -04:00
parent 175b412317
commit aef0990835
2 changed files with 4 additions and 6 deletions

View File

@ -631,7 +631,7 @@ void RDLibraryModel::updateModel(const QString &filter_sql)
return;
}
#ifdef RDLIBRARYMODEL_ENABLE_UPDATE_PROFILING
printf("%p - filter_sql: %s\n",this,filter_sql.toUtf8().constData());
// printf("%p - filter_sql: %s\n",this,filter_sql.toUtf8().constData());
RDTimeProbe *probe=new RDTimeProbe();
probe->printWaypoint("updateModel - 1");
#endif // RDLIBRARYMODEL_ENABLE_UPDATE_PROFILING
@ -674,7 +674,9 @@ void RDLibraryModel::updateModel(const QString &filter_sql)
#endif // RDLIBRARYMODEL_ENABLE_UPDATE_PROFILING
sql=sqlFields()+
filter_sql;
filter_sql+
QString::asprintf(" limit %d ",RD_LIMITED_CART_SEARCH_QUANTITY);
beginResetModel();
d_texts.clear();
d_notes.clear();
@ -691,7 +693,6 @@ void RDLibraryModel::updateModel(const QString &filter_sql)
probe->printWaypoint("updateModel - 4");
#endif // RDLIBRARYMODEL_ENABLE_UPDATE_PROFILING
// printf("RDLibraryModel::updateModel() SQL: %s\n",sql.toUtf8().constData());
q=new RDSqlQuery(sql);
while(q->next()&&(carts_loaded<d_cart_limit)) {
if(q->value(0).toUInt()!=prev_cartnum) {
@ -707,7 +708,6 @@ void RDLibraryModel::updateModel(const QString &filter_sql)
prev_cartnum=q->value(0).toUInt();
carts_loaded++;
}
// printf("carts_loaded: %d\n",carts_loaded);
}
delete q;

View File

@ -69,8 +69,6 @@ EditEvent::EditEvent(QString eventname,bool new_event,QStringList *new_events,
event_lib_view->setModel(event_lib_model);
event_lib_view->hideColumn(3);
event_cart_filter->setModel(event_lib_model);
connect(event_cart_filter,SIGNAL(filterChanged(const QString &,int)),
event_lib_model,SLOT(setFilterSql(const QString &,int)));
connect(rda->ripc(),SIGNAL(userChanged()),
event_cart_filter,SLOT(changeUser()));
connect(event_lib_view->selectionModel(),