Ok, had some others patching their Redmine installations to fix this same issue and obviously I did something stupid to mess up the patch (to be honest, maybe the 'fix' is dumb too, but it's working for us and we're goal-oriented, heh).

Here's a proper one I think and this one also preemptively fixes the other l(msg) calls in error_messages_for() :

<pre>
--- broken/app/helpers/application_helper.rb 2009-01-30 16:07:47.000000000 -0600
+++ app/helpers/application_helper.rb 2009-01-30 16:03:36.000000000 -0600
@@ -524,11 +525,11 @@
full_messages = []
object.errors.each do |attr, msg|
next if msg.nil?
- msg = msg.first if msg.is_a? Array
+ msg = [msg] if(msg.is_a?(String))
if attr == "base"
- full_messages << l(msg)
+ full_messages << l(*msg)
else
- full_messages << "&#171; " + (l_has_string?("field_" + attr) ? l("field_" + attr) : object.class.human_attribute_name(attr)) + " &#187; " + l(msg) unless attr == "custom_values"
+ full_messages << "&#171; " + (l_has_string?("field_" + attr) ? l("field_" + attr) : object.class.human_attribute_name(attr)) + " &#187; " + l(*msg) unless attr == "custom_values"
end
end
# retrieve custom values error messages
@@ -536,8 +537,8 @@
object.custom_values.each do |v|
v.errors.each do |attr, msg|
next if msg.nil?
- msg = msg.first if msg.is_a? Array
- full_messages << "&#171; " + v.custom_field.name + " &#187; " + l(msg)
+ msg = [msg] if(msg.is_a?(String))
+ full_messages << "&#171; " + v.custom_field.name + " &#187; " + l(*msg)
end
end
end
</pre>

--------------------------------------------------------------------------------
It was reported in #2200 but I'm not able to reproduce this problem using ruby 1.8.7/rails 2.1.2.
Which language are you using? Did you made any change to this lang file or to other file?
--------------------------------------------------------------------------------

I see you are also using Ruby 1.8.7 and Rails 2.1.2 like we are.

We haven't changed any language files or a language setting. The default is English, right? Not being familiar, I'm guessing it's using <code>lang/en.yml</code> as a source of message Strings as part of localization. And the GLoc library to effect that localization.

Redmine was obtained via: <code>"svn co svn://rubyforge.org/var/svn/redmine/trunk ."</code>

This included <code>vendor/plugins/gloc-1.1.0/</code>

I believe I saw the other bug and maybe another, but they didn't seem to recognize that the code was causing _any_ message with format fields (%d, %f, etc) that gets translated via GLoc's <code>l()</code> method to throw an exception. So it was a much broader problem.

I was able to replicate their bug and others like it with Ruby 1.8.7 and Rails 2.1.2. I was thinking folks seeing this problem might like to have a "fix" to the code that has worked for us. We put the "fix" in our Redmine install notes, since any Redmine install we did had this problem.

I don't suppose you have some other version of GLoc in addition to the one in <code>vendor/</code>, perhaps that works a little differently?

Or perhaps your <code>error_message_for()</code> in application_helper.rb hasn't got the offending <code>"msg = msg.first if msg.is_a? Array"</code> lines?

I see that <code>lang/en.yml</code> has the relevant messages:
* <code>text_caracters_maximum: %d characters maximum.</code>
* <code>text_caracters_minimum: Must be at least %d characters long.</code>

Those are going to throw exceptions if <code>l(msg)</code> is called after doing <code>"msg = msg.first if msg.is_a? Array"</code>. When msg was an Array, it had the message String _and_ the appropriate argument to fill the %d. My so-called fix just makes sure <code>l()</code> receives both the String and the arguments to fill it (and thus not crash).

Thanks for taking a look, Jean-Philippe.
I have to admit that we just started trying out Redmine and it's great! Also I have to admit that I'm not familiar with Rails, but I can read code and make stuff work (maybe not the right way, but _a_ way)
--------------------------------------------------------------------------------
Thanks for your feedback.
I understand why the missing arguments raise an error but I can't see why it occurs.

The 2 strings you mention (text_caracters_maximum and text_caracters_minimum) are never used to display validation error messages. They're just used to display tips on various form and they're always called with the appropriate argument.

The validation errors use these string instead:
* @activerecord_error_too_long: is too long@
* @activerecord_error_too_short: is too short@

And as you can see, there's no parameter to supply.
--------------------------------------------------------------------------------

Hi Jean-Philippe,

Jean-Philippe Lang wrote:
> The 2 strings you mention (text_caracters_maximum and text_caracters_minimum) are never used to display validation error messages. They're just used to display tips on various form and they're always called with the appropriate argument.
>
> The validation errors use these string instead:
> * @activerecord_error_too_long: is too long@
> * @activerecord_error_too_short: is too short@

Hmm, I see...no, it's the 2nd part of the message that is the problem. Let me find two I know used to cause exceptions. Here we go:

The 'too short' message from Change Password form:

<code>« Password » is too short (minimum is 4 characters)</code>

The too long and a too short message from Create New Project Form:

<code>« Name » is too long (maximum is 30 characters)<br>
« Identifier » can't be blank<br>
« Identifier » is too short (minimum is 2 characters)</code>

It is the "maximum" and "minimum" part of the messages that cause the error. Debugging shows that these have %d where the integers go, but no arguments to fill the %d are available in <code>GLoc._l()</code>

> And as you can see, there's no parameter to supply.

Yes, I see that. Um, where does the 2nd half of the message come from? Is that a standard Rails thing I'm not familiar with, but that is being retrieved as <code>msg</code> by the iterator <code> object.errors.each do |attr, msg|</code> in <code>error_messages_for()</code>? Because debugging shows that <code>msg</code> is an Array containing the String and a numeric argument to fill the format field (such as 4, 30, etc).

Thanks,
Andrew
--------------------------------------------------------------------------------
AFAIK, "is too short (minimum is 4 characters)" is a default Rails 2.1 error message.
It should be overriden by source:/trunk/config/initializers/10-patches.rb.

Which gives:

!too_short_error.png!
--------------------------------------------------------------------------------
Jean-Philippe Lang wrote:
> AFAIK, "is too short (minimum is 4 characters)" is a default Rails 2.1 error message.
> It should be overriden by source:/trunk/config/initializers/10-patches.rb.

The 10-patches.rb we have looks like that, minus the AsynchronousMailer code you added 2 days ago (Revision 2339).

Although, I don't understand since it says:

<code>
10 :too_long => "activerecord_error_too_long", <br>
11 :too_short => "activerecord_error_too_short",
</code>

Which, to my ignorant eye, seems to indicate to use the messages that came with activerecord? How is that doing the overriding (I'm getting out of my depth here...I'm better at figuring out and fixing problems than talking about how Rails apps work).

I can verify that the ActiveRecord I got when installing Rails 2.1.2 gives me:

<code>
/ruby/gems/1.8/gems/activerecord-2.1.2/lib/active_record/validations.rb:34: :too_long => "is too long (maximum is %d characters)",
</code>

Which is what I'm seeing [now, with my fix; before it crashed], and you are not...I don't seem to have the overriding you do, even though the <code>ActiveRecord::Errors.default_error_messages</code> Hash in 10-patches.rb matches what you linked to. I wonder what I'm missing.

Question: isn't the default Rails "too short" message more informative and isn't it better to have that displayed? The fix I've put in, for example, doesn't break messages with no format fields and takes advantage of <code>l()</code> and <code>_l()</code> having an argument glob.

From this respect, I kind of like the <code>« Password » is too short (minimum is 4 characters)</code> I've got now, rather than a less informative version. Although maybe I've introduced some other problem...and why don't I see what you see sort of bothers me.

--------------------------------------------------------------------------------
> Which, to my ignorant eye, seems to indicate to use the messages that came with activerecord?

Not exactly, "activerecord_error_too_short" is not the actual error message (as you can see it's not the same as @:too_long => "is too long (maximum is %d characters)"@ that you find in rails code). It's the key that is used to get the translated error message when doing @l(msg)@ in @ApplicationHelper#error_messages_for@ that overrides validation error messages display.

These hacks will be removed in Redmine 0.9 since Rails 2.2 has builtin support for internationalization :-)
But I can apply your patch for 0.8.x releases.
--------------------------------------------------------------------------------

Oh, duh, I see "activerecord_error_too_short" etc in my ./lang/en.yml with the values you mentioned. And in 10-patches.rb you've told it to use those keys and thus those values. Ok, that makes sense, I should have seen that.

I'm still confused why our installs would not be using your values--while I like the messages I have now, it suggests I've skipped some step or config setting, or I've got something wrong. When does 10-patches.rb and such get applied; every start-up of the app or when one does a db:migrate or something else?

Thanks for all the feedback and explanations, I appreciate it, Jean-Philippe.
--------------------------------------------------------------------------------
Fix applied in r2486.
The reason why this error occurs is still unknown.
--------------------------------------------------------------------------------
Merged in 0.8 branch in r2513.
--------------------------------------------------------------------------------

duplicates,Closed,2787,validation fails when project name exceeds 30 characters on create