tdabek@go2.pl pisze:
A.L.E.C pisze:
tzn. ma to być gotowy patch i najlepiej jakby umożliwiał włączenie tego przy pomocy opcji konfiguracyjnej (w dodatku wyklikiwanej z menu Userpanel), a wtedy nie widzę problemu aby go wrzucić do cvsu na stałe.
jesli opcja wyklikiwana to chyba musi byz dodatkowe pole w bazie?
nie musi, zobacz jak są zrobione opcje w konfiguracji Userpanela (menu Userpanel)
Chodzi o to, że nie zawsze mam ochotę przeglądać i poprawiać patche, których z resztą za wiele nie ma.
skoro nie ma za wiele to...
zauważ że nikt mi nie płaci za przeglądanie patchy, jeśli jakiś mnie zainteresuje i mam czas to to robię, niestety zazwyczaj muszę je poprawiać, żeby były "w stylu LMSa", nie mówiąc o usuwaniu błędów. Poza tym jest dość łatwo zostać deweloperem, aby samemu wrzucać zmiany do cvsu, ale nie ukrywam, że zdarzyło się też iż odebrałem uprawnienia za wrzucanie wadliwego kodu.
P.S. Jesli posiadam juz funkcjonalnosc, za ktora zaplacilem ekstra, wyswietlania na karcie klienta jego daty logowania do UP to nie moge tego udostepnic dla wszystkich za free?
możesz
wiec wrzuc to prosze do CVS no chya ze tez musi byc opcja wyklikiwana.
Napisałem że możesz udostępnić, a nie że można dodać do cvsu (to też uzasadniłem). Nawet jeśli ten patch nadawałby się do contrib, to i tak nie mam zamiaru dbać o jego aktualność.