Does this list accept attachments? It got rejected as spam :)

Rainer Hermanns wrote:
No idea, just zip together the newly created dir and I'll add the files
individually :)

tia,
Rainer

You are right, the patch doesn't include the new files inside the added
folder, I can't figure out how to make svn create a patch with the added
folder, it keeps complaining:

"Files inside the added folder won't be included ...."

Am I doing something wrong or this is a svn/patch limitation?

musachy

Rainer Hermanns wrote:
Ah, okay... So it looks like the complete folder with all moved widget
files
is missing. The patch does not create the folder, nor does it create the
"new/moved" template files.

thanks for your help,
Rainer


I will take a look tomorrow (not on my box right now), but I do
remember
that I changed the widgets to struts.widget from struts.widgets, for 2
reasons, 1. Keep consistency with Dojo (not that we had to anyway) and
2.
Dojo 0.4 by default assumes the widgets to be on a subdir "widget"
which
requires less configuration. I will let you know  tomorrow.

thanks

musachy

On 11/5/06, Rainer Hermanns <[EMAIL PROTECTED]> wrote:

Musachy,

after applying your patch, all custom widget files do no longer have
content.
Is it correct, that all files under core/src/main/resources/org/
apache/struts2/static/dojo/struts/widget/ should be removed?

If yes, shouldn't core/src/main/resources/template/ajax/
dojoRequire.js should be changed like this (remove all lines with '-'
):

dojo.require("dojo.io.BrowserIO");
dojo.require("dojo.event.topic");

- dojo.hostenv.setModulePrefix('struts', 'struts');
dojo.require('dojo.widget.*');
- dojo.widget.manager.registerWidgetPackage('struts.widget');

- dojo.require("struts.widget.Bind");
- dojo.require("struts.widget.BindDiv");
- dojo.require("struts.widget.BindAnchor");
dojo.require("dojo.widget.Editor");
dojo.hostenv.writeIncludes(); // not needed, but allows the Venkman
debugger to work with the includes

It looks to me, that either the patch is not complete, or that I am
missing the part replacing the above widget requirements.

Could you please have a look at this?

Thanks,
Rainer

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



--
"Hey you! Would you help me to carry the stone?" Pink Floyd


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to