On the module page:

> will use the standard File Field behaviour and take the maximum file size from the upload

AFA I can tell, Filefield doesn't use upload module limits at all, just the PHP limit.
See this for example: #150393: Upload limit settings are not respected

Comments

joachim’s picture

To be a bit less brief, my point is that the current text is misleading -- using upload module's per role limits is an extra feature of this module, and the project page should say so.

greg.harvey’s picture

Hmm, a feature I wasn't aware of! Cool! =)

Still not quite sure I get it though? Could you write the lines you think should be added/changed in here and I'll paste them in to the project page? I think if you write it fully I'll get what you mean...

joachim’s picture

Well, we're saying:

> will use the standard File Field behaviour and take the maximum file size from the upload

And taking the max size isn't standard FF behaviour.

> will use the standard File Field behaviour
means "use PHP upload limit"
which is not the same as:
> take the maximum file size from the upload

So we're equating two unequal things.

Correct version is to either not mention the standard FF:

> you can simply leave it as 0 and it will take the maximum file size from the upload module settings.

Or to say we do it differently:

Correct version is to either not mention the standard FF:

> you can simply leave it as 0 and it will take the maximum file size from the upload module settings (note this is unlike FF default behaviour which uses only the PHP upload limit).

greg.harvey’s picture

Status: Active » Fixed

Gotcha. Strange I thought it worked differently. That text is about 3 years old now, I think, so maybe FF has changed. Anyway, fixed and thanks. =)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.