The Book Solution Pack's 'delete' button says 'Delete Book' rather than just 'Delete'; this allows the Book Solution Pack to use its own special label, as well as any other solution packs that might use different labels in the future.
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
It's a bit leaky in that it makes assumptions about the members that will exist
on AbstractObjects, but we can't make cloning work at the moment as it doesn't
really work well with inheritance, every decendent would have to define __clone
functions properly and I figured a wrapper function like this would be easier
to support in the long term, until we make the movement to decorated objects.
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.
Nigel Banks