Looks like there was a thinko in 8a7d34b such that a select field was using the name for the id even when an `$id` argument was passed. So I've fixed it to use the `$id` and to fallback on the name when the `$id` isn't set. [#111 state:resolved]. There seems to be an unrelated issue where submitting a search in the contributor popup by hitting "enter" instead of the "search" button causes any contributors already added to be forgotten. What happens is the "enter" submits the contributor's "Delete" button instead of the "Search" buttton. This is really irritating. I'm not sure how to fix it, though; I'll create a new bug to track it.
prevents the SOAP server from having to guess the SQL type of a field type when updating an element type. Thanks to David Oliveira for the report [#79 state:resolved].
…<list()> method of Bric::Biz::Asset::Business::Story, Bric::Biz::Asset::Business::Media, and Bric::Biz::Asset::Template now works as expected, returning the latest version of all stories regardless of their publish status, rather than no stories at all (Bug #1).
Fixed compatibility issues on MySQL. Bricolage now sets `sql_mode` when it connects to the database, so that MySQL itself does not have to globally set that option, as it could cause conflicts with other apps running on MySQL. Also removed the attempt to set C<sql_mode> globally at installation time; it was bad manners, really. I used the DBI's callback support to make this work, so bumped up the minimum required DBI to 1.49. That's five years old already, so everyone should be fine.
* Added `unescape_uri()` to Bric::Util::ApacheUtil. * Updated the various URI-processing methods in Bric::Util::Trans::FS to properly escape or unescape URIs as appropriate. * Wrote tests for the URI-processing methods in Bric::Util::Trans::FS. * Fixed `PREVIEW_LOCAL` block in Bric::Util::Burner to convert a URI to a directory with the improved `uri_to_dir()` method in Bric::Util::Trans::FS. * Updated the distribution movers (FS, FTP, and SFTP) to convert from URIs to file-system paths. Left WebDAV alone, since it should actually use URIs.
…yer is not tall enough #88
Fixed a bug in `find_or_create_alternate()` where an element type is passed with a different setting for its `top_level` attribute than is the value of the `top_level` attribute of the original image's element type. This caused problems because it looked for an existing image using the wrong URI. So I hacked in a workaround to fool the current image into using the alternat's element type when determining the URI, so that the URI should always be the same. I ran into this in production for The Tyee, where editors were trying to create non-fixed thumbnails for fixed images. This caused some pretty serious indigestion. The workaround is a bit of a hack, but it's clean and works well. I updated the test suite to replicate the issue before patching, so we do have a proper regression test for the issue.
I updated the docs for the new `mark_as_published()` method in `Bric::Biz::Asset::Business`, removing some invalid notes, adding some valid notes, and fixing some grammar nits. I also removed the call to `save()` at the end of the method, because no method should call `save()`; the user should always call it implicitly. I also changed the method so that it simply returns if the document is already marked as published, as there is no point in updating the publish date in this context. And finally, I added some tests for the new method.