View Issue Details

WikiRelated ChangesetsJump to Notes
IDProjectCategoryView StatusDate SubmittedLast Update
0013272mantisbtinstallationpublic2011-08-26 09:342014-09-23 18:05
ReporterCarstenGrohmann Assigned Todregad  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.2.6 
Target Version1.2.8Fixed in Version1.2.8 
Summary0013272: Hardcoded table name mantis_config_table in install.php
Description

Hi,

I installed the Oracle branch yesterday. This branch contains a minor bug.
An insert statement uses a hard coded config table name:

echo 'INSERT INTO mantis_config_table ( value, ...

it could be (like the main trunk):
echo 'INSERT INTO ' . db_get_table( 'config' ) . ' ( value, ...

Please change this single line...

Regards,
Carsten

TagsNo tags attached.

Relationships

related to 0015721 closedgrangeway Functionality to consider porting to master-2.0.x 
related to 0013227 closeddregad Oracle DB support multiple issues 

Activities

CarstenGrohmann

CarstenGrohmann

2011-08-26 09:45

reporter   ~0029563

Probably "db_get_table( 'mantis_config_table' )" is the better statement.
rombert

rombert

2011-08-28 09:32

reporter   ~0029568

Thank you for the report and suggested fix. Would you consider submitting pull requests for this functionality? This would greatly increase the speed of including these changes in MantisBT.

Ideally you would submit pull requests for both the master and master-1.2.x branches at https://github.com/mantisbt/mantisbt .
grangeway

grangeway

2011-08-28 09:58

reporter   ~0029570

This issue has already been fixed in the latest trunk
dregad

dregad

2011-08-29 09:23

developer   ~0029592

It was fixed on master only, not in 1.2.x.
grangeway

grangeway

2013-04-05 17:57

reporter   ~0036406

Marking as 'acknowledged' not resolved/closed to track that change gets ported to master-2.0.x branch

Related Changesets

MantisBT: master-1.2.x b4af2384

2011-08-29 00:58

dregad


Details Diff		 Fix 0013272: Hardcoded table name mantis_config_table in install.php

This issue applies only to 1.2.x, as it had already been fixed in master as
part of commit c8f355d04fb5120545d890ea2a7b2e2b6afc9b6c		 Affected Issues
0013272
mod - admin/install.php Diff File