Hi, Gareth! First of all, nice work! NHibernate is indeed in need of some supporting tools; NHProf and Visual NHibernate are good examples of such tools! Some comments, however:
- Why aren't all properties virtual by default, or, at least, those properties that represent relations to other entities? I also noticed that they are indeed all created as virtual, despite what appears in the editor; - When generating a model from the DB, when representing a many-to- many relationship, there is no need to map the middle table, unless, of course, this table has extra attributes other than the mapping keys, so why include it in the model? - Why aren't other databases supported, other than Microsoft's? What about SQL Server 2008? OK, I noticed that you can use SQL Server 2008 if you choose SQL Server 2005, but IMO it should be explicit, like, "SQL Server 2005/2008"; - Classes should be marked with the Serializable attribute (or even with WCF's DataContract and DataMember), it doesn't cause any harm, and allows them to be serialized; perhaps it could be an option? - No support for lazyness yet; - No support for private setters for properties; - No support (at least, I haven't found it) for setting id generator type; - No proper support for composite keys: instead of generating a property for each key, perhaps it would be better to generate an inner class that would wrap these properties, and present a single key object, since all NHibernate's APIs expect a single object as a key. Please note that all this is just my humble opinion! :-) Keep up the good work! RP On Dec 2, 10:54 am, Gareth <[email protected]> wrote: > Fabio, you'll be pleased to know that screenshots have now been added. > Videos might take a bit longer ;-) > > See them here:http://www.slyce.com/VisualNHibernate/Screenshots/ > > Gareth. > > On Dec 2, 4:35 pm, Gareth <[email protected]> wrote: > > > > > We have just uploaded a new build which you can grab > > here:http://www.slyce.com/downloads/ > > > Diego, your bug has been fixed - thanks for reporting it. > > > cws, your suggestion has been implemented - thanks for suggesting it. > > > There are a few other little fixes as well. > > > Cheers, > > Gareth. > > > On Dec 2, 6:45 am, Gareth <[email protected]> wrote: > > > > Oh yes it would....we'll see what we can do ;-) > > > > Gareth. > > > > On Dec 1, 11:39 pm, cws <[email protected]> wrote: > > > > > Nice tool! > > > > > Feeback: > > > > > In the reference edit pane, wouldn't it be nice to see which columns > > > > the foreign-keys belongs to? > > > > > On 1 Dec, 10:56, Gareth <[email protected]> wrote: > > > > > > Thanks for reporting this Diego. We'll get a fix out asap. > > > > > > Gareth. > > > > > > On Dec 1, 10:46 pm, Diego Mijelshon <[email protected]> wrote: > > > > > > > Reverse engineering doesn't seem to work well... I tried to open my > > > > > > "DataAccess" project (which contains the mappings and > > > > > > hibernate.cfg.xml) and > > > > > > got: > > > > > > --------------------------- > > > > > > An Error Occurred > > > > > > --------------------------- > > > > > > Could not find the NHibernate *.cfg.xml in your project > > > > > > --------------------------- > > > > > > > Diego > > > > > > > On Mon, Nov 30, 2009 at 23:40, Gareth <[email protected]> > > > > > > wrote: > > > > > > > We have just released a new version of the Visual NHibernate beta > > > > > > > and > > > > > > > are looking for some testers to provide feeedback. VNH can > > > > > > > reverse- > > > > > > > engineer existing nHibernate projects, allowing you to visualize > > > > > > > your > > > > > > > mappings etc. This new version also supports basic NH Validator > > > > > > > usage. > > > > > > > > It doesn't yet have 100% coverage of the hbm schema, but it will > > > > > > > let > > > > > > > report what elements it is not expecting. Please let us know what > > > > > > > we're currently missing, and how we can improve this tool for you. > > > > > > > > We are tring to reduce the learning-curve for nHibernate to enable > > > > > > > more people to start using nHibernate in their projects. > > > > > > > > Thanks, > > > > > > > Gareth Hayter. > > > > > > > Slyce Software. > > > > > > > > -- > > > > > > > > You received this message because you are subscribed to the > > > > > > > Google Groups > > > > > > > "nhusers" group. > > > > > > > To post to this group, send email to [email protected]. > > > > > > > To unsubscribe from this group, send email to > > > > > > > [email protected]<nhusers%[email protected] > > > > > > > > > > > > > > > . > > > > > > > For more options, visit this group at > > > > > > >http://groups.google.com/group/nhusers?hl=en. -- You received this message because you are subscribed to the Google Groups "nhusers" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/nhusers?hl=en.
