joniles / mpxj

Primary repository for MPXJ library

Home Page:http://www.mpxj.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Primavera XML - Relationship Lag Calendar

rsinha9 opened this issue · comments

It looks like relationship lag calendar is defined on the Project for Primavera XML files exported out of OPC which is correctly defined in the schema. It just is not mapped over to the CustomProperties since the CustomProperties is only dependent on ScheduleOptions.

Two Options:

  1. I can update the ProjectProperties object to have a RelationshipLagCalendar with an enum on it and map it over.
  2. I can update to map from the project level property into the CustomProperties

Let me know which one you prefer and I can implement and get a PR over to you.

I've implemented option #1, and I left the custom properties there. Let me know if you want any changes. Happy to make them.

@rsinha9 I'm (eventually) going to stop using custom properties for P6 configuration options, they'll all have their own attributes on the project, so option 2 is the best approach. I'll take a look at implementing this myself over the weekend, and will also add support read/write support for XER files.

Of course when I said option 2 above, what I actually meant was option 1. It's late here...

It looks like you've implemented the change, I'll take a look.

Of course when I said option 2 above, what I actually meant was option 1. It's late here...

Not your fault, I preferred Option 2 so I made it Option 1 in an edit

It looks like you've implemented the change, I'll take a look.

Yup, did it for both read/write. I left the original custom property there, but happy to remove it as well. I also added tests for all of them.

Thanks again for the PR - changes merged in #576

I've just published a new release which contains these changes.