will work and where it will not.
This test failed due to an API change in WebGUI. Before 7.3, if you created
a Storage object and then created an Asset with that Storage Id, then WebGUI
would do that happily. This is actually a bug, because you could create multiple
Assets with that one Storage Id, and when _any_ of the Assets was deleted or
rolled back it would delete the Storage location and leave the other Assets
with that Id pointing to an empty Storage object.
So the API change is actually a bug fix, albeit being a side-effect of the
auto-commit work for posts and wikis.
of processTemplate, there is a call to $self->get which adds all the asset
properties to the list of template variables. Sometimes they duplicate
entries in the www_ method (like view). Sometimes they're not interesting,
like templateId, but they're still valid template variables.
I'm going to start a new template variable entry for each asset that
will just contain those variables. Any www_ method calling processTemplate
can then include them via ISA into their list of template variables.
Fixed a typo in the Macro_r_printable where an ending bold tag was dropped.
Add some POD to Asset.pm talking about variables that are added when
processTemplate is called.
More ISA work in the Help system.
Broke out code scanning tests into their own tests, and required the CODE_COP environment
variable be set before they were run.
Fixed two bugs in the Workflow test, wrong number of tests and inverse test logic.
More Auth template inheritance work. Almost time to reflect it over to
the LDAP module.
by both AuthWebGUI and AuthLDAP. I've pulled the defaults out of
AuthWebGUI for one template (anonymous registration) into its own
namspace, Auth. Default variables will be pulled out for all of the templates
eventually.
Minor streamlining of profile field listing methods.
WebGUI::Operation::Profile::getRequiredProfileFields is now deprecated,
because it's specific enough to be not significantly more useful than
WebGUI::ProfileField::getRequiredFields, and because with this change
nothing in core uses it anymore.
It didn't work with CIDR format IP addresses. I cannabilized the
code from canShowDebug and refactored it into its own method,
canShowBasedOnIP. Both canShowDebug and canShowPerformanceIndicators call this.
Updated the docs for the debugIp setting to note that it's used for both
showDebug and showPerformanceIndicators.
Added about 20 more tests to ErrorHandler.t. Only the two hard methods are
left, fatal (which closes the session at the end) and showDebug, which is
validating a ton of output.
