Bug #11032

Regression: DIP upload not working in 2.4

Added by Dan Gillean over 3 years ago. Updated almost 3 years ago.

Status:VerifiedStart date:04/03/2017
Priority:HighDue date:
Assignee:Kelly Stewart% Done:

0%

Category:Archivematica integration
Target version:Release 2.4.0
Google Code Legacy ID: Tested version:2.4
Sponsored:No Requires documentation:

Description

Reported by Santi in the lead-up to Archivematica Camp York:

PHP Fatal error:  Call to a member function getAclGroups() on a non-object in /usr/share/nginx/atom/lib/myUser.class.php on line 82

Santi temporarily resolved the issue by commenting out the following lines in lib/myUser.class.php:

/*
     foreach ($user->getAclGroups() as $group)
     {
       $this->addCredential($group->getName(array('culture' => 'en')));
     }

*/

Santi's original notes are on internal issue #1102. Note that I don't think we encountered this issue during AtoM Camp SFU, but not sure if the AM demo was new or an internal site.

History

#1 Updated by Nick Wilkinson over 3 years ago

  • Assignee changed from Nick Wilkinson to Mike Gale

#3 Updated by Dan Gillean about 3 years ago

  • Priority changed from Medium to High

#4 Updated by Nick Wilkinson about 3 years ago

  • Assignee changed from Mike Gale to José Raddaoui Marín

#5 Updated by José Raddaoui Marín about 3 years ago

  • Status changed from New to Code Review
  • Assignee changed from José Raddaoui Marín to Mike Gale

Ready for code review, PR 556

#6 Updated by José Raddaoui Marín about 3 years ago

  • Assignee changed from Mike Gale to Nick Wilkinson

#7 Updated by Mike Gale about 3 years ago

  • Assignee changed from Nick Wilkinson to José Raddaoui Marín

Looks good to me!

#8 Updated by José Raddaoui Marín about 3 years ago

  • Status changed from Code Review to Deploy
  • Assignee changed from José Raddaoui Marín to Nick Wilkinson

Merged in qa/2.4.x, the AtoM worker needs to be restarted.

#9 Updated by Nick Wilkinson about 3 years ago

  • Status changed from Deploy to Feedback
  • Assignee changed from Nick Wilkinson to Santiago Collazo

Hi Santi, do you think this should be deployed for any AtoM camp VMs that are configured for receiving DIPs?

#10 Updated by David Juhasz about 3 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from Santiago Collazo to Dan Gillean

Dan, please QA/Review. Kelly says you have to.

#11 Updated by Dan Gillean about 3 years ago

  • Assignee deleted (Dan Gillean)

#12 Updated by Kelly Stewart about 3 years ago

  • Assignee set to Kelly Stewart

#13 Updated by Kelly Stewart almost 3 years ago

  • Status changed from QA/Review to Verified

Also available in: Atom PDF