NanoContainer
  1. NanoContainer
  2. NANO-132

hibernate3 module upgrade for hb3-FINAL VERSION! THERE IS API CHANGE!

          Details

          • Type: Improvement Improvement
          • Status: Closed Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: None
          • Component/s: None
          • Labels:
            None
          • Number of attachments :
            3

            Description

            Here it goes an zipped version and a patch version of the hibernate3 nanocontainer module.

              Issue Links

              relates to

              Improvement - An improvement or enhancement to an existing feature or task. NANO-130 hibernate3 module updated to hibernate3-rc1 (api changed)

              • Major - Major loss of function.
              • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

                Activity

                Jose Peleteiro made changes -
                Field Original Value New Value
                Attachment hibernate3.patch [ 14197 ]
                Hide
                Permalink
                Jose Peleteiro added a comment -

                This is the rigth project.xml file. Sorry about it.

                Show
                Jose Peleteiro added a comment - This is the rigth project.xml file. Sorry about it.
                Jose Peleteiro made changes -
                Attachment project.xml [ 14198 ]
                Aslak Hellesøy made changes -
                Link This issue is related to NANO-130 [ NANO-130 ]
                Aslak Hellesøy made changes -
                Comment [ 31511 ]
                Hide
                Permalink
                Aslak Hellesøy added a comment -

                What do we do with NANO-130? Does this patch supersede that one?

                As described in NANO-130, your patch should be in a patch format:

                I'm not keen on adding all of these classes that only seem to be minor modification of the original org.nanocontainer.hibernate package, except that your classes are in a different package - the maintenance burden of 2 quasi-similar packages seems too high. (Correct me if I'm wrong).

                Could you refactor your classes to be in the same package and submit a patch? Aslo see: http://picocontainer.org/How+To+Contribute

                Show
                Aslak Hellesøy added a comment - What do we do with NANO-130 ? Does this patch supersede that one? As described in NANO-130 , your patch should be in a patch format: I'm not keen on adding all of these classes that only seem to be minor modification of the original org.nanocontainer.hibernate package, except that your classes are in a different package - the maintenance burden of 2 quasi-similar packages seems too high. (Correct me if I'm wrong). Could you refactor your classes to be in the same package and submit a patch? Aslo see: http://picocontainer.org/How+To+Contribute
                Hide
                Permalink
                Jose Peleteiro added a comment -

                This is a patch for the cvs version.

                Its package is org.nanocontainer.hibernate3 cause:

                • hb2 and hb3 Session interface is very diferent.
                • It could never use hb2 and hb3 together, what is needed in some case, like mine.
                • If you create nano-persistence (which I understand is going to put hb2 and hb3 together), you'll get problems.
                Show
                Jose Peleteiro added a comment - This is a patch for the cvs version. Its package is org.nanocontainer.hibernate3 cause: hb2 and hb3 Session interface is very diferent. It could never use hb2 and hb3 together, what is needed in some case, like mine. If you create nano-persistence (which I understand is going to put hb2 and hb3 together), you'll get problems.
                Hide
                Permalink
                Jose Peleteiro added a comment -

                Can I do anything to help? Im very interested about nanocontainer-hibernate3.

                I can help, Im already a hausmate (login name "juze") and owner of xfire-picocontainer, can I help to get it updated and rocking?

                Cheers,
                Jose Peleteiro

                Show
                Jose Peleteiro added a comment - Can I do anything to help? Im very interested about nanocontainer-hibernate3. I can help, Im already a hausmate (login name "juze") and owner of xfire-picocontainer, can I help to get it updated and rocking? Cheers, Jose Peleteiro
                Hide
                Permalink
                Jörg Schaible added a comment -

                Patience, patience! Hibernate is mainly Konstantin's child, so we expect him to comment on this. Although he is normally very responsive, he seems currently to be under load.

                Show
                Jörg Schaible added a comment - Patience, patience! Hibernate is mainly Konstantin's child, so we expect him to comment on this. Although he is normally very responsive, he seems currently to be under load.
                Konstantin Pribluda made changes -
                Assignee Konstantin Pribluda [ ko5tik ]
                Hide
                Permalink
                Konstantin Pribluda added a comment -

                I applied patches, please verify that eveyrthing is correct

                Show
                Konstantin Pribluda added a comment - I applied patches, please verify that eveyrthing is correct
                Konstantin Pribluda made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Konstantin Pribluda made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

                  • Assignee:
                    Konstantin Pribluda
                    Reporter:
                    Jose Peleteiro
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    1 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: