joniles / mpxj

Primary repository for MPXJ library
http://www.mpxj.org/
GNU Lesser General Public License v2.1
248 stars 104 forks source link

Primavera Import Fails on WBS Processing. #551

Closed amarkssmartpm closed 1 year ago

amarkssmartpm commented 1 year ago

If no activities under a WBS entry contain an actual start date the process will fail when calculating the duration. In my case there is one activity which is a finish milestone. This type of milestone does not contain a actual start date.

amarkssmartpm commented 1 year ago

I added a code change for this on my fork, amarkssmartpm/mpxj.

joniles commented 1 year ago

PR: https://github.com/joniles/mpxj/pull/552

joniles commented 1 year ago

Would you be able to add a sample file here, or to the PR, to replicate the issue. It'll save me some time constructing something appropriate - thanks!

amarkssmartpm commented 1 year ago

Here you go.

On Fri, Aug 18, 2023 at 6:46 AM Jon Iles @.***> wrote:

Would you be able to add a sample file here, or to the PR, to replicate the issue. It'll save me some time constructing something appropriate - thanks!

— Reply to this email directly, view it on GitHub https://github.com/joniles/mpxj/issues/551#issuecomment-1683726608, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALOEA5HKOZVUN5K4JJOOCALXV5BX5ANCNFSM6AAAAAA3TBQAFI . You are receiving this because you authored the thread.Message ID: @.***>

-- [image: SmartPM Logo] Andrew Marks

Senior Software Engineer | SmartPM Technologies 404.329.3000 | 770.977.8328 @.*** | www.smartpmtech.com

[image: -] https://linkedin.com/company/smartpmtech [image: -] https://twitter.com/SmartPMTech [image: -] @.***> [image: -] https://facebook.com/SmartPMTech Follow us on LinkedIn https://www.linkedin.com/company/smartpmtech

-- Confidentiality Notice:This email and any attachments it may contain are intended for the individual or entity above and may be confidential and protected by legal privilege. SmartPM Technologies, Inc. makes no warranties, express or implied, concerning the accuracy of the information contained herein, which is subject to change without notice. Unintended recipients should be aware that use of this e-mail or any of its attachments in any way is forbidden.

joniles commented 1 year ago

@amarkssmartpm unfortunately I think the sample file may have been stripped off during the email->GitHub comment translation. Would you be able to post it here directly? Thanks!

amarkssmartpm commented 1 year ago

badxml.zip

amarkssmartpm commented 1 year ago

Attached the zip file with the xml.

amarkssmartpm commented 1 year ago

I attached in the issue.

On Sun, Aug 20, 2023 at 4:46 AM Jon Iles @.***> wrote:

@amarkssmartpm https://github.com/amarkssmartpm unfortunately I think the sample file may have been stripped off during the email->GitHub comment translation. Would you be able to post it here directly? Thanks!

— Reply to this email directly, view it on GitHub https://github.com/joniles/mpxj/issues/551#issuecomment-1685227533, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALOEA5FOCQ42D7DRPCJXWM3XWHFIFANCNFSM6AAAAAA3TBQAFI . You are receiving this because you were mentioned.Message ID: @.***>

-- [image: SmartPM Logo] Andrew Marks

Senior Software Engineer | SmartPM Technologies 404.329.3000 | 770.977.8328 @.*** | www.smartpmtech.com

[image: -] https://linkedin.com/company/smartpmtech [image: -] https://twitter.com/SmartPMTech [image: -] @.***> [image: -] https://facebook.com/SmartPMTech Follow us on LinkedIn https://www.linkedin.com/company/smartpmtech

-- Confidentiality Notice:This email and any attachments it may contain are intended for the individual or entity above and may be confidential and protected by legal privilege. SmartPM Technologies, Inc. makes no warranties, express or implied, concerning the accuracy of the information contained herein, which is subject to change without notice. Unintended recipients should be aware that use of this e-mail or any of its attachments in any way is forbidden.

joniles commented 1 year ago

Thank you! I've merged your change.