Re: [dev] Antw: Re: manual file change result in package error

2014-07-03 Thread Josef1 Penzkofer
Hi Peter,

we use OTRS 3.2.14.

I copied several template-files to
/Custom/Kernel/Output/HTML/Standard
and modified this. This still works on our system.

I only had problems with a modified *.pm-file, that I copied into 
/Custom/Kernel/Modules.

But I must admit, I'm not sure, wether I restarted the webserver Apache. 

I think, in general - the mechanism to copy files into /Custom/Kernel/... and
modify the files still works - works also with template-files.

Jo

>>> Peter Krall  03.07.2014 15:04 >>>
Apologies, I meant Templates and not Skins, but other than that, it should
work under Output\HTML\CustomTemplate folder.
Peter


On Thu, Jul 3, 2014 at 3:01 PM, Peter Krall  wrote:

> Hi Renée,
>
> are you sure custom Skins (Output\HTML) should go under Custom? It never
> actually worked for me this way either, and sounds as if it would go head
> to head against what the dev doc says about Skins:
> http://otrs.github.io/doc/manual/developer/3.3/en/html/skins.html 
>
> I have a modified custom Skin under Kernel\Output\HTML\ name> and it works very nicely. It did not work when putting the custom
> DTLs under Custom\Kernel\Output\HTML\Standard.
>
> Of course I had to create the required XML config entries before the Skin
> was usable (as explained in the above doc).
>
> Cheers,
> Peter
>
>
> On Wed, Jul 2, 2014 at 5:28 PM, Renee B 
> wrote:
>
>> Hi Jo,
>>
>>
>> On 02.07.2014 17:12, Josef1 Penzkofer wrote:
>> > Hi Renée,
>> >
>> > and this works properly?
>>
>> yes. This should even work in OTRS 3.2.x (maybe not for templates, I
>> can't remember when that was fixed).
>>
>> > Yesterday I wrote an E-Mail to "otrs...@otrs.org" because I made
>> changes in
>> > file "CustomerTicketMessage.pm".
>> >
>> > Therefore I copied the file in directory "/Custom/Kernel/Modules".
>> >
>> > The changes does not work.
>>
>> Did you restart the Apache? Did you check if the "original" file was
>> loaded? Did you add some debug statements?
>>
>> > The changes works, after I made the changes into file
>> > "CustomerTicketMessage.pm"; directory: /Kernel/Modules.
>> >
>> > Strange: I copied some other files into directory "/Custom/Kernel/...
>> and made
>> > changes. This changes works.
>> >
>> > So, whats going wrong?
>>
>> I'll have a look at the source code tomorrow. Maybe there is a
>> difference how Customer related modules  are loaded compared to the
>> modules related to the Agent interface.
>>
>> > Thanks in advance.
>> >
>> > Jo
>>
>> - Renée
>>
>> --
>> Perl / OTRS development: http://perl-services.de 
>> OTRS AddOn repository: http://opar.perl-services.de 
>>
>> ___
>> OTRS mailing list: dev - Webpage: http://otrs.org/ 
>> Archive: http://lists.otrs.org/pipermail/dev 
>> To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev 
>>
>
>
___
OTRS mailing list: dev - Webpage: http://otrs.org/
Archive: http://lists.otrs.org/pipermail/dev
To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev

Re: [dev] Antw: Re: manual file change result in package error

2014-07-03 Thread Renee B
On 03.07.2014 15:01, Peter Krall wrote:
> Hi Renée,
>
> are you sure custom Skins (Output\HTML) should go under Custom? It never
> actually worked for me this way either, and sounds as if it would go head
> to head against what the dev doc says about Skins:
> http://otrs.github.io/doc/manual/developer/3.3/en/html/skins.html

As you already said, Templates != Skins.

And yes, this should work (I already used that several times).

These are the lines in the code responsible for that feature:

https://github.com/OTRS/otrs/blob/rel-3_3/Kernel/Output/HTML/Layout.pm#L398
https://github.com/OTRS/otrs/blob/rel-3_3/Kernel/Output/HTML/Layout.pm#L539

https://github.com/OTRS/otrs/blob/rel-3_3/Kernel/Config/Defaults.pm#L813

See this commit:
https://github.com/OTRS/otrs/commit/4cdcff91ff8cab98d6bd351b9096714a6e490d19

I can't say why it didn't worked for you. That would need some debugging...
(Was there an old Defaults.pm or old Layout.pm? Were the correct
permissions set etc.? What version of OTRS was that? Was the config
rebuilt? Was the server restartet? ...)

Kind regards,
Renée

-- 
Perl / OTRS development: http://perl-services.de
OTRS AddOn repository: http://opar.perl-services.de
___
OTRS mailing list: dev - Webpage: http://otrs.org/
Archive: http://lists.otrs.org/pipermail/dev
To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev


Re: [dev] Antw: Re: manual file change result in package error

2014-07-03 Thread Peter Krall
Apologies, I meant Templates and not Skins, but other than that, it should
work under Output\HTML\CustomTemplate folder.
Peter


On Thu, Jul 3, 2014 at 3:01 PM, Peter Krall  wrote:

> Hi Renée,
>
> are you sure custom Skins (Output\HTML) should go under Custom? It never
> actually worked for me this way either, and sounds as if it would go head
> to head against what the dev doc says about Skins:
> http://otrs.github.io/doc/manual/developer/3.3/en/html/skins.html
>
> I have a modified custom Skin under Kernel\Output\HTML\ name> and it works very nicely. It did not work when putting the custom
> DTLs under Custom\Kernel\Output\HTML\Standard.
>
> Of course I had to create the required XML config entries before the Skin
> was usable (as explained in the above doc).
>
> Cheers,
> Peter
>
>
> On Wed, Jul 2, 2014 at 5:28 PM, Renee B 
> wrote:
>
>> Hi Jo,
>>
>>
>> On 02.07.2014 17:12, Josef1 Penzkofer wrote:
>> > Hi Renée,
>> >
>> > and this works properly?
>>
>> yes. This should even work in OTRS 3.2.x (maybe not for templates, I
>> can't remember when that was fixed).
>>
>> > Yesterday I wrote an E-Mail to "otrs...@otrs.org" because I made
>> changes in
>> > file "CustomerTicketMessage.pm".
>> >
>> > Therefore I copied the file in directory "/Custom/Kernel/Modules".
>> >
>> > The changes does not work.
>>
>> Did you restart the Apache? Did you check if the "original" file was
>> loaded? Did you add some debug statements?
>>
>> > The changes works, after I made the changes into file
>> > "CustomerTicketMessage.pm"; directory: /Kernel/Modules.
>> >
>> > Strange: I copied some other files into directory "/Custom/Kernel/...
>> and made
>> > changes. This changes works.
>> >
>> > So, whats going wrong?
>>
>> I'll have a look at the source code tomorrow. Maybe there is a
>> difference how Customer related modules  are loaded compared to the
>> modules related to the Agent interface.
>>
>> > Thanks in advance.
>> >
>> > Jo
>>
>> - Renée
>>
>> --
>> Perl / OTRS development: http://perl-services.de
>> OTRS AddOn repository: http://opar.perl-services.de
>>
>> ___
>> OTRS mailing list: dev - Webpage: http://otrs.org/
>> Archive: http://lists.otrs.org/pipermail/dev
>> To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev
>>
>
>
___
OTRS mailing list: dev - Webpage: http://otrs.org/
Archive: http://lists.otrs.org/pipermail/dev
To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev

Re: [dev] Antw: Re: manual file change result in package error

2014-07-03 Thread Peter Krall
Hi Renée,

are you sure custom Skins (Output\HTML) should go under Custom? It never
actually worked for me this way either, and sounds as if it would go head
to head against what the dev doc says about Skins:
http://otrs.github.io/doc/manual/developer/3.3/en/html/skins.html

I have a modified custom Skin under Kernel\Output\HTML\
and it works very nicely. It did not work when putting the custom DTLs
under Custom\Kernel\Output\HTML\Standard.

Of course I had to create the required XML config entries before the Skin
was usable (as explained in the above doc).

Cheers,
Peter


On Wed, Jul 2, 2014 at 5:28 PM, Renee B  wrote:

> Hi Jo,
>
>
> On 02.07.2014 17:12, Josef1 Penzkofer wrote:
> > Hi Renée,
> >
> > and this works properly?
>
> yes. This should even work in OTRS 3.2.x (maybe not for templates, I
> can't remember when that was fixed).
>
> > Yesterday I wrote an E-Mail to "otrs...@otrs.org" because I made
> changes in
> > file "CustomerTicketMessage.pm".
> >
> > Therefore I copied the file in directory "/Custom/Kernel/Modules".
> >
> > The changes does not work.
>
> Did you restart the Apache? Did you check if the "original" file was
> loaded? Did you add some debug statements?
>
> > The changes works, after I made the changes into file
> > "CustomerTicketMessage.pm"; directory: /Kernel/Modules.
> >
> > Strange: I copied some other files into directory "/Custom/Kernel/...
> and made
> > changes. This changes works.
> >
> > So, whats going wrong?
>
> I'll have a look at the source code tomorrow. Maybe there is a
> difference how Customer related modules  are loaded compared to the
> modules related to the Agent interface.
>
> > Thanks in advance.
> >
> > Jo
>
> - Renée
>
> --
> Perl / OTRS development: http://perl-services.de
> OTRS AddOn repository: http://opar.perl-services.de
>
> ___
> OTRS mailing list: dev - Webpage: http://otrs.org/
> Archive: http://lists.otrs.org/pipermail/dev
> To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev
>
___
OTRS mailing list: dev - Webpage: http://otrs.org/
Archive: http://lists.otrs.org/pipermail/dev
To unsubscribe: http://lists.otrs.org/cgi-bin/listinfo/dev