From noreply at atlassian.com Fri Apr 24 21:10:43 2009
Content-Type: multipart/mixed; boundary="===============8655282780842551613=="
MIME-Version: 1.0
From: Michael Sabin (JIRA) <noreply at atlassian.com>
To: hibernate-issues at lists.jboss.org
Subject: [hibernate-issues] [Hibernate-JIRA] Created: (HBX-1123) Reveng
 foreign-key map to non-primary key column produces incorrect mapping file
Date: Fri, 24 Apr 2009 20:10:17 -0500
Message-ID: <1307389134.1240621817232.JavaMail.j2ee_opensource.atlassian.com-projects@atlassian12.managed.contegix.com>

--===============8655282780842551613==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

Reveng foreign-key map to non-primary key column produces incorrect mapping=
 file
---------------------------------------------------------------------------=
-----

                 Key: HBX-1123
                 URL: http://opensource.atlassian.com/projects/hibernate/br=
owse/HBX-1123
             Project: Hibernate Tools
          Issue Type: Bug
    Affects Versions: 3.2.4 Beta1
         Environment: Eclipse Platform
Version: 3.4.2
Build id: M20090211-1700

Hibernate Tools 3.2.4.GA-R200903141626-H5

Windows XP Pro SP3

JRE 6

Tested on Oracle 10g and Apache Derby
            Reporter: Michael Sabin
            Priority: Minor
         Attachments: CodeGenCfg1.png, CodeGenCfg2.png

Creating a mapping from a column in one table to a non-primary key column i=
n another table produces incorrect mapping file missing a property-ref to t=
he non-primary key.

How to reproduce using Apache Derby:

Create a database and connect to the database.
jdbc:derby://localhost:1527/myDB;create=3Dtrue;user=3Dme;password=3Dmine

Create these tables in the myDB database, in the ME schema.

create table vehicles (
	make_id integer, =

	model_id integer,
	name varchar(50),
	PRIMARY KEY (make_id, model_id)
);

create table owners (
	owner_id integer PRIMARY KEY,
	favorite_make_id integer,
	name varchar(20)
);

-- optional content
insert into vehicles values (1, 10, 'Chevrolet Corvette');
insert into vehicles values (2, 11, 'Ford Focus');
insert into vehicles values (3, 12, 'Honda Accord');
insert into vehicles values (4, 13, 'Toyota Camry');

insert into owners values (50, 3, 'Jim');
insert into owners values (51, 1, 'Alex');
insert into owners values (52, 4, 'Sue');
insert into owners values (53, 1, 'Casey');


Create this Hibernate Configuration file.

<?xml version=3D"1.0" encoding=3D"UTF-8"?>
<!DOCTYPE hibernate-configuration PUBLIC
		"-//Hibernate/Hibernate Configuration DTD 3.0//EN"
		"http://hibernate.sourceforge.net/hibernate-configuration-3.0.dtd">
<hibernate-configuration>
    <session-factory>
        <property name=3D"hibernate.connection.driver_class">org.apache.der=
by.jdbc.EmbeddedDriver</property>
        <property name=3D"hibernate.connection.url">jdbc:derby://localhost:=
1527/myDB</property>
        <property name=3D"hibernate.connection.username">me</property>
        <property name=3D"hibernate.connection.password">mine</property>
        <property name=3D"hibernate.dialect">org.hibernate.dialect.DerbyDia=
lect</property>
	    <property name=3D"hibernate.current_session_context_class">thread</pro=
perty>
        <property name=3D"hibernate.default_schema">ME</property>
	    <property name=3D"hibernate.show_sql">true</property>
    </session-factory>
</hibernate-configuration>


Setup a Hibernate Console Configuration with the configuration above and co=
nnection to the Derby database.

Create this Hibernate reverse engineering configuration

<?xml version=3D"1.0" encoding=3D"UTF-8"?>
<!DOCTYPE hibernate-reverse-engineering PUBLIC "-//Hibernate/Hibernate Reve=
rse Engineering DTD 3.0//EN" "http://hibernate.sourceforge.net/hibernate-re=
verse-engineering-3.0.dtd" >

<hibernate-reverse-engineering>
	<table-filter match-name=3D"OWNERS"/>
	<table-filter match-name=3D"VEHICLES"/>
  =

	<table name=3D"VEHICLES">
		<foreign-key foreign-table=3D"OWNERS">
			<column-ref local-column=3D"MAKE_ID" foreign-column=3D"FAVORITE_MAKE_ID"=
 />
		</foreign-key>
	</table>
  =

</hibernate-reverse-engineering>


Setup a Hibernate Code Generation configuration like in the attached images.

Running the Code Generation configuration will produce this Vehicles.hbm.xm=
l mapping file.

<?xml version=3D"1.0"?>
<!DOCTYPE hibernate-mapping PUBLIC "-//Hibernate/Hibernate Mapping DTD 3.0/=
/EN"
"http://hibernate.sourceforge.net/hibernate-mapping-3.0.dtd">
<hibernate-mapping>
    <class name=3D"reveng.Vehicles" table=3D"VEHICLES">
        <composite-id name=3D"id" class=3D"reveng.VehiclesId">
            <key-property name=3D"makeId" type=3D"int">
                <column name=3D"MAKE_ID" />
            </key-property>
            <key-property name=3D"modelId" type=3D"int">
                <column name=3D"MODEL_ID" />
            </key-property>
        </composite-id>
        <many-to-one name=3D"owners" class=3D"reveng.Owners" update=3D"fals=
e" insert=3D"false" fetch=3D"select">
            <column name=3D"MAKE_ID" not-null=3D"true" />
        </many-to-one>
        <property name=3D"name" type=3D"string">
            <column name=3D"NAME" length=3D"50" />
        </property>
    </class>
</hibernate-mapping>

The many-to-one tag should have this additional attribute: property-ref=3D"=
favoriteMakeId"

-- =

This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: htt=
p://opensource.atlassian.com/projects/hibernate/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       =20

--===============8655282780842551613==--

From noreply at atlassian.com Thu Jul 12 09:10:36 2012
Content-Type: multipart/mixed; boundary="===============7074770644559292458=="
MIME-Version: 1.0
From: =?utf-8?q?Hrotk=C3=B3_G=C3=A1bor_=28JIRA=29_=3Cnoreply_at_atlassian=2Ecom?=
 =?utf-8?q?=3E?=
To: hibernate-issues at lists.jboss.org
Subject: [hibernate-issues] [Hibernate-JIRA] (HBX-1123) Reveng foreign-key map
 to non-primary key column produces incorrect mapping file
Date: Thu, 12 Jul 2012 08:10:36 -0500
Message-ID: <637949482.3887.1342098636269.JavaMail.j2ee-onjira@hibernate.onjira.com>
In-Reply-To: 1307389134.1240621817232.JavaMail.j2ee_opensource.atlassian.com-projects@atlassian12.managed.contegix.com

--===============7074770644559292458==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable


--===============7074770644559292458==
Content-Type: text/html
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="attachment.html"

CgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKPHN0eWxlPgovKiBDaGFuZ2luZyB0aGUgbGF5b3V0
IHRvIHVzZSBsZXNzIHNwYWNlIGZvciBtb2JpbGVzICovCkBtZWRpYSBzY3JlZW4gYW5kIChtYXgt
ZGV2aWNlLXdpZHRoOiA0ODBweCksIHNjcmVlbiBhbmQgKC13ZWJraXQtbWluLWRldmljZS1waXhl
bC1yYXRpbzogMikgewogICAgI2VtYWlsLWJvZHkgeyBtaW4td2lkdGg6IDMwZW0gIWltcG9ydGFu
dDsgfQogICAgI2VtYWlsLXBhZ2UgeyBwYWRkaW5nOiA4cHggIWltcG9ydGFudDsgfQogICAgI2Vt
YWlsLWJhbm5lciB7IHBhZGRpbmc6IDhweCA4cHggMCA4cHggIWltcG9ydGFudDsgfQogICAgI2Vt
YWlsLWF2YXRhciB7IG1hcmdpbjogMXB4IDhweCA4cHggMCAhaW1wb3J0YW50OyBwYWRkaW5nOiAw
ICFpbXBvcnRhbnQ7IH0KICAgICNlbWFpbC1maWVsZHMgeyBwYWRkaW5nOiAwIDhweCA4cHggOHB4
ICFpbXBvcnRhbnQ7IH0KICAgICNlbWFpbC1ndXR0ZXIgeyB3aWR0aDogMCAhaW1wb3J0YW50OyB9
Cn0KPC9zdHlsZT4KPGRpdiBpZD0iZW1haWwtYm9keSI+Cjx0YWJsZSBpZD0iZW1haWwtd3JhcCIg
YWxpZ249ImNlbnRlciIgYm9yZGVyPSIwIiBjZWxscGFkZGluZz0iMCIgY2VsbHNwYWNpbmc9IjAi
IHN0eWxlPSJiYWNrZ3JvdW5kLWNvbG9yOiNmMGYwZjA7Y29sb3I6IzAwMDAwMDt3aWR0aDoxMDAl
OyI+CiAgICA8dHIgdmFsaWduPSJ0b3AiPgogICAgICAgIDx0ZCBpZD0iZW1haWwtcGFnZSIgc3R5
bGU9InBhZGRpbmc6MTZweCAhaW1wb3J0YW50OyI+CiAgICAgICAgICAgIDx0YWJsZSBhbGlnbj0i
Y2VudGVyIiBib3JkZXI9IjAiIGNlbGxwYWRkaW5nPSIwIiBjZWxsc3BhY2luZz0iMCIgc3R5bGU9
ImJhY2tncm91bmQtY29sb3I6I2ZmZmZmZjtib3JkZXI6MXB4IHNvbGlkICNiYmJiYmI7Y29sb3I6
IzAwMDAwMDt3aWR0aDoxMDAlOyI+CiAgICAgICAgICAgICAgICA8dHIgdmFsaWduPSJ0b3AiPgog
ICAgICAgICAgICAgICAgICAgIDx0ZCBiZ2NvbG9yPSIjM2U0YzRlIiBzdHlsZT0iYmFja2dyb3Vu
ZC1jb2xvcjojM2U0YzRlO2NvbG9yOiNmZmZmZmY7Zm9udC1mYW1pbHk6QXJpYWwsRnJlZVNhbnMs
SGVsdmV0aWNhLHNhbnMtc2VyaWY7Zm9udC1zaXplOjEycHg7bGluZS1oZWlnaHQ6MTsiPjxpbWcg
c3JjPSJodHRwczovL3d3dy5qYm9zcy5vcmcvZG1zL2hpYmVybmF0ZS9pbWFnZXMvamlyYS9qaXJh
aGVhZGVyX2hpYmVybmF0ZS5wbmciIGFsdD0iIiBzdHlsZT0idmVydGljYWwtYWxpZ246dG9wOyIg
Lz48L3RkPgogICAgICAgICAgICAgICAgPC90cj48dHIgdmFsaWduPSJ0b3AiPgogICAgPHRkIGlk
PSJlbWFpbC1iYW5uZXIiIHN0eWxlPSJwYWRkaW5nOjMycHggMzJweCAwIDMycHg7Ij4KCiAgICAg
ICAgICAgICAgICAKICAgICAgICAKICAgICAgICAKICAgICAgICAgICAgPHRhYmxlIGFsaWduPSJs
ZWZ0IiBib3JkZXI9IjAiIGNlbGxwYWRkaW5nPSIwIiBjZWxsc3BhY2luZz0iMCIgd2lkdGg9IjEw
MCUiIHN0eWxlPSJ3aWR0aDoxMDAlOyI+CiAgICA8dHIgdmFsaWduPSJ0b3AiPgogICAgICAgIDx0
ZCBzdHlsZT0iY29sb3I6IzUwNTA1MDtmb250LWZhbWlseTpBcmlhbCxGcmVlU2FucyxIZWx2ZXRp
Y2Esc2Fucy1zZXJpZjtwYWRkaW5nOjA7Ij4KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgIDxpbWcgaWQ9ImVtYWlsLWF2YXRhciIgc3JjPSJodHRwczovL2hpYmVybmF0ZS5v
bmppcmEuY29tL3NlY3VyZS91c2VyYXZhdGFyP2F2YXRhcklkPTEwMTYyIiBhbHQ9IiIgaGVpZ2h0
PSI0OCIgd2lkdGg9IjQ4IiBib3JkZXI9IjAiIGFsaWduPSJsZWZ0IiBzdHlsZT0icGFkZGluZzow
O21hcmdpbjogMCAxNnB4IDE2cHggMDsiIC8+CiAgICAgICAgICAgICAgICAgICAgICAgIDxkaXYg
aWQ9ImVtYWlsLWFjdGlvbiIgc3R5bGU9InBhZGRpbmc6IDAgMCA4cHggMDtmb250LXNpemU6MTJw
eDtsaW5lLWhlaWdodDoxOHB4OyI+CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
IDxhIGNsYXNzPSJ1c2VyLWhvdmVyIiByZWw9InJvdGkiIGlkPSJlbWFpbF9yb3RpIiBocmVmPSJo
dHRwczovL2hpYmVybmF0ZS5vbmppcmEuY29tL3NlY3VyZS9WaWV3UHJvZmlsZS5qc3BhP25hbWU9
cm90aSIgc3R5bGU9ImNvbG9yOiM2Yzc5N2Y7Ij5Icm90a8OzIEfDoWJvcjwvYT4KICAgICBjb21t
ZW50ZWQgb24gPGltZyBzcmM9Imh0dHBzOi8vaGliZXJuYXRlLm9uamlyYS5jb20vaW1hZ2VzL2lj
b25zL2J1Zy5naWYiIGhlaWdodD0iMTYiIHdpZHRoPSIxNiIgYm9yZGVyPSIwIiBhbGlnbj0iYWJz
bWlkZGxlIiBhbHQ9IkJ1ZyI+IDxhIHN0eWxlPSdjb2xvcjojNmM3OTdmO3RleHQtZGVjb3JhdGlv
bjpub25lOycgaHJlZj0naHR0cHM6Ly9oaWJlcm5hdGUub25qaXJhLmNvbS9icm93c2UvSEJYLTEx
MjMnPkhCWC0xMTIzPC9hPgogICAgICAgICAgICA8L2Rpdj4KICAgICAgICAgICAgICAgICAgICAg
ICAgPGRpdiBpZD0iZW1haWwtc3VtbWFyeSIgc3R5bGU9ImZvbnQtc2l6ZToxNnB4O2xpbmUtaGVp
Z2h0OjIwcHg7cGFkZGluZzoycHggMCAxNnB4IDA7Ij4KICAgICAgICAgICAgICAgIDxhIHN0eWxl
PSdjb2xvcjojNmM3OTdmO3RleHQtZGVjb3JhdGlvbjpub25lOycgaHJlZj0naHR0cHM6Ly9oaWJl
cm5hdGUub25qaXJhLmNvbS9icm93c2UvSEJYLTExMjMnPjxzdHJvbmc+UmV2ZW5nIGZvcmVpZ24t
a2V5IG1hcCB0byBub24tcHJpbWFyeSBrZXkgY29sdW1uIHByb2R1Y2VzIGluY29ycmVjdCBtYXBw
aW5nIGZpbGU8L3N0cm9uZz48L2E+CiAgICAgICAgICAgIDwvZGl2PgogICAgICAgICAgICAgICAg
ICAgIDwvdGQ+CiAgICA8L3RyPgo8L3RhYmxlPgogICAgPC90ZD4KPC90cj4KPHRyIHZhbGlnbj0i
dG9wIj4KICAgIDx0ZCBpZD0iZW1haWwtZmllbGRzIiBzdHlsZT0icGFkZGluZzowIDMycHggMzJw
eCAzMnB4OyI+CiAgICAgICAgPHRhYmxlIGJvcmRlcj0iMCIgY2VsbHBhZGRpbmc9IjAiIGNlbGxz
cGFjaW5nPSIwIiBzdHlsZT0icGFkZGluZzowO3RleHQtYWxpZ246bGVmdDt3aWR0aDoxMDAlOyIg
d2lkdGg9IjEwMCUiPgogICAgICAgICAgICA8dHIgdmFsaWduPSJ0b3AiPgogICAgICAgICAgICAg
ICAgPHRkIGlkPSJlbWFpbC1ndXR0ZXIiIHN0eWxlPSJ3aWR0aDo2NHB4O3doaXRlLXNwYWNlOm5v
d3JhcDsiPjwvdGQ+CiAgICAgICAgICAgICAgICA8dGQ+CiAgICAgICAgICAgICAgICAgICAgPHRh
YmxlIGJvcmRlcj0iMCIgY2VsbHBhZGRpbmc9IjAiIGNlbGxzcGFjaW5nPSIwIiB3aWR0aD0iMTAw
JSI+CiAgICAgICAgICAgICAgICAgICAgICAgIDx0ciB2YWxpZ249InRvcCI+CiAgICA8dGQgY29s
c3Bhbj0iMiIgc3R5bGU9ImNvbG9yOiMwMDAwMDA7Zm9udC1mYW1pbHk6QXJpYWwsRnJlZVNhbnMs
SGVsdmV0aWNhLHNhbnMtc2VyaWY7Zm9udC1zaXplOjEycHg7cGFkZGluZzowIDAgMTZweCAwO3dp
ZHRoOjEwMCU7Ij4KICAgICAgICA8ZGl2IGNsYXNzPSJjb21tZW50LWJsb2NrIiBzdHlsZT0iYmFj
a2dyb3VuZC1jb2xvcjojZWRmNWZmO2JvcmRlcjoxcHggc29saWQgI2RkZGRkZDtjb2xvcjojMDAw
MDAwO3BhZGRpbmc6MTJweDsiPjxwPkkgdGhpbmssIHRoYXQgdGhlIGVqYjMgbWFwcGluZyBpcyBh
bHNvIGFmZmVjdGVkIGJ5IHRoaXM6PC9wPgoKPHA+aXQgd2lsbCBnZW5lcmF0ZTo8L3A+Cgo8cD5A
TWFueVRvT25lKGZldGNoPUZldGNoVHlwZS5MQVpZKTxici8+CkBKb2luQ29sdW1uKG5hbWU9Ik1B
S0VfSUQiLCBudWxsYWJsZT1mYWxzZSk8L3A+Cgo8cD5pbnN0ZWFkIG9mOjwvcD4KCjxwPkBNYW55
VG9PbmUoZmV0Y2g9RmV0Y2hUeXBlLkxBWlkpPGJyLz4KQEpvaW5Db2x1bW4obmFtZT0iTUFLRV9J
RCIsIG51bGxhYmxlPWZhbHNlLCByZWZlcmVuY2VkQ29sdW1uTmFtZT0iRkFWT1JJVEVfTUFLRV9J
RCIpPC9wPgoKCjxwPnZlcnNpb246IEhpYmVybmF0ZSAzLjMuMS5HQTwvcD48L2Rpdj4KICAgICAg
ICA8ZGl2IHN0eWxlPSJjb2xvcjojNTA1MDUwO3BhZGRpbmc6NHB4IDAgMCAwOyI+ICAgICAgICAg
ICAgICAgIDwvZGl2PgogICAgPC90ZD4KPC90cj4KICAgICAgICAgICAgICAgICAgICA8L3RhYmxl
PgogICAgICAgICAgICAgICAgPC90ZD4KICAgICAgICAgICAgPC90cj4KICAgICAgICA8L3RhYmxl
PgogICAgPC90ZD4KPC90cj4KCgoKCgoKCgoKCgoKCiAgICAgICAgICAgIDwvdGFibGU+CiAgICAg
ICAgPC90ZD48IS0tIEVuZCAjZW1haWwtcGFnZSAtLT4KICAgIDwvdHI+CiAgICA8dHIgdmFsaWdu
PSJ0b3AiPgogICAgICAgIDx0ZCBzdHlsZT0iY29sb3I6IzUwNTA1MDtmb250LWZhbWlseTpBcmlh
bCxGcmVlU2FucyxIZWx2ZXRpY2Esc2Fucy1zZXJpZjtmb250LXNpemU6MTBweDtsaW5lLWhlaWdo
dDoxNHB4O3BhZGRpbmc6IDAgMTZweCAxNnB4IDE2cHg7dGV4dC1hbGlnbjpjZW50ZXI7Ij4KICAg
ICAgICAgICAgVGhpcyBtZXNzYWdlIGlzIGF1dG9tYXRpY2FsbHkgZ2VuZXJhdGVkIGJ5IEpJUkEu
PGJyIC8+CiAgICAgICAgICAgIElmIHlvdSB0aGluayBpdCB3YXMgc2VudCBpbmNvcnJlY3RseSwg
cGxlYXNlIGNvbnRhY3QgeW91ciA8YSBzdHlsZT0nY29sb3I6IzZjNzk3ZjsnIGhyZWY9J2h0dHBz
Oi8vaGliZXJuYXRlLm9uamlyYS5jb20vc2VjdXJlL0NvbnRhY3RBZG1pbmlzdHJhdG9ycyFkZWZh
dWx0LmpzcGEnPkpJUkEgYWRtaW5pc3RyYXRvcnM8L2E+LjxiciAvPgogICAgICAgICAgICBGb3Ig
bW9yZSBpbmZvcm1hdGlvbiBvbiBKSVJBLCBzZWU6IDxhIHN0eWxlPSdjb2xvcjojNmM3OTdmOycg
aHJlZj0naHR0cDovL3d3dy5hdGxhc3NpYW4uY29tL3NvZnR3YXJlL2ppcmEnPmh0dHA6Ly93d3cu
YXRsYXNzaWFuLmNvbS9zb2Z0d2FyZS9qaXJhPC9hPgogICAgICAgIDwvdGQ+CiAgICA8L3RyPgo8
L3RhYmxlPjwhLS0gRW5kICNlbWFpbC13cmFwIC0tPgo8L2Rpdj48IS0tIEVuZCAjZW1haWwtYm9k
eSAtLT4K

--===============7074770644559292458==--