Also includes some general clean up.
* Namespaces no longer uses ajax now uses states.
* Abstracted out repository access message into a seperate function.
* Simplified the code.
Tests ingest forms by using a shared variable 'models'.
Additional stuff:
'models' is no longer a required config for ingest steps, now
its set from the given objects.
No longer need to rebuild steps from within a form step when the
shared_storage gets changed, we now detect if changes have been
made then we rebuild if necessary.
Fixed POST requests in WebTestCases previously the user was always
anonymous so interactions with fedora would fail via post requests.
This reverts commit 992c42b96f1faed5fc48eb1dd8680537b1f05831...
... Apparently, the "production" code review has a bit of a bug in it,
where it doesn't parse @code/@endcode sections properly:
http://drupal.org/node/1991382
There was a mistake where a step was reverting its global changes before it was
actually selected as the previous step. Innocuous enough but it had strange
side effects on the global storage, since the step list would be rebuild with
this information, steps could insert themselves as previous steps even if they
weren't used by the user.
May my code be clean, and follow the law's of "standard".
May it support internationalization for Travis includes all!
May it not contain copy pasta for it is soggy.
Travis be praised!
The message will only be displayed to the user if Drupal is configured to show these
warnings, regardless of the setting though it will be logged to the watchdog.
Fix's an error in my logic. Now instead of having a null step ID mean
that it is the first step. The step ID gets initialized when the
form is first build and NULL is changed to mean that all steps
have been executed.
Also added some comments.
nruest