You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@roller.apache.org by Michael Bien <mb...@gmail.com> on 2021/06/02 15:21:59 UTC
Re: Splitting the struts.xml file
Hi Yash,
i actually don't mind central configs - at least until they grow large
and become difficult to maintain.
It has currently ~650 lines which isn't too bad IMO.
There is also an alternative to splitting: a lot of it could be moved to
annotations which would shrink the file.
i would probably leave it as is until it grows but I am not opposed to
the idea to cleaning things up,
best regards,
michael
On 29.05.21 11:51, Yash Maheshwari wrote:
> Hi Team,
>
> I was exploring the roller project and found that the struts.xml file
> contains mappings for all the packages in it. But I think we can split this
> file into multiple XML files each one representing a single package. And
> then including those in the struts.xml file.
>
> This will help in finding and understanding a specific package mapping
> easily and will make the file more understandable.
>
> WDYT?
>
> Thank You,
> Yash Maheshwari
>
Re: Splitting the struts.xml file
Posted by Yash Maheshwari <ya...@gmail.com>.
Hi,
That makes sense, Michael.
Thank You,
Yash Maheshwari
On Wed, 2 Jun 2021 at 20:56, Michael Bien <mb...@gmail.com> wrote:
> Hi Yash,
>
> i actually don't mind central configs - at least until they grow large
> and become difficult to maintain.
> It has currently ~650 lines which isn't too bad IMO.
>
> There is also an alternative to splitting: a lot of it could be moved to
> annotations which would shrink the file.
>
>
> i would probably leave it as is until it grows but I am not opposed to
> the idea to cleaning things up,
>
> best regards,
> michael
>
>
> On 29.05.21 11:51, Yash Maheshwari wrote:
> > Hi Team,
> >
> > I was exploring the roller project and found that the struts.xml file
> > contains mappings for all the packages in it. But I think we can split
> this
> > file into multiple XML files each one representing a single package. And
> > then including those in the struts.xml file.
> >
> > This will help in finding and understanding a specific package mapping
> > easily and will make the file more understandable.
> >
> > WDYT?
> >
> > Thank You,
> > Yash Maheshwari
> >
>
>