Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0012779
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] A. Platformminorhave not tried2010-03-23 19:292022-02-01 08:08
ReportershuehnerView Statuspublic 
Assigned ToTriage Platform Base 
PrioritynormalResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0012779: 'Create columns from db' should warn or skip column with names >30 as those are not allowed

DescriptionColumn with length or the name >30 chars don't work in oracle so erp has a validation to warn about them.

However 'Create column from db' happily creates those without warnings and only later on validation/export you get a note and need to rename all columns again.

The ad_table_import process should either skip and warn or just warn about those columns directly.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0025690)
camcal (reporter)
2010-03-23 20:32

Actually the issue occurs when you try to install a module that has a column with more than 30 characters, on another instance of OB.

During the rebuild it says that the column name has more than 30 character, but during the development it says "ERROR=value too long for type character varying(40)" using a column with more than 30 characters.

This happens when you try to create columns using "Create Columns from DB" button, and the column has more than 40 characters.

Also if you try to manually create a column, it allows you to type a name up to 40 characters long without warnings.

I want to emphasize that the 30 characters error occurs during the rebuild, not on the export, update, package or compile.development tasks.
(0025698)
shuehner (administrator)
2010-03-24 12:10

Hi camcal,

i can't agree completely as current version of openbravo do the 30 character name length validation when exporting/packaging a module, only some older MP's did not yet have this check.

With that in place the installation parts gets less scary as you get told when exporting/packaging modules/updates already.
(0025735)
camcal (reporter)
2010-03-25 17:45

Hi huehner.

I would say that the validation must be at the moment of the creation of the columns, so the developer can make the respective modifications before using them on a window, tab, alert, etc.

If you create a few columns, let's say 20, and you use them for filtering purposes, you will have to wait until the export/package task to realize that this columns doesn't fullfill the restrictions.

That means that the developer will have to rename all the columns and make the respective modifications on every single place that they are being used.

I'm telling you this, due the problems that i faced for this issue.

I had to made several modifications to my module replacing the names on every single place that i used them.
(0031993)
stalker (reporter)
2010-10-19 23:09

(a weak voice from community crowd)

May be its worth considering to make a switch for developers?
something like... "Allow >30 chars for relnames" or... "Oracle rules!" )
Its always developer's (implementor's or partner's) decision which platform to utilize or consider... so why make all around send curses to Oracle?... )

- Issue History
Date Modified Username Field Change
2010-03-23 19:29 shuehner New Issue
2010-03-23 19:29 shuehner Assigned To => alostale
2010-03-23 20:17 camcal Issue Monitored: camcal
2010-03-23 20:32 camcal Note Added: 0025690
2010-03-24 12:10 shuehner Note Added: 0025698
2010-03-25 17:45 camcal Note Added: 0025735
2010-07-27 16:00 rafaroda Issue Monitored: rafaroda
2010-10-19 23:09 stalker Note Added: 0031993
2017-04-10 14:39 alostale Assigned To alostale => platform
2022-02-01 08:08 alostale Assigned To platform => Triage Platform Base


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker