Lejárati idő dátummal

[juhasz]

[barnabas-nagy]

Órán ez a feladat nem lett elkezdve? Nulláról kell indulni? (legutóbb sajnos nem voltam.)

[juhasz]

jelenleg egy legördülő menüből lehet kiválasztani, hogy mennyi idő múlva járjon le:
https://github.com/dCourse/shrtr/blob/3c8e72c8519e08af6012d4250fbd19a3a5c12141/shrtr.module#L38-48

ezt egy timestamp-ként menti el

shrtr/shrtr.module

Line 79 in 3c8e72c

$expire = REQUEST_TIME + $form_state['values']['expire'];

az elképzelésem itt az volna, hogy felveszel egy újabb dátum field-et, és ha ott beállít valamit, akkor az lesz a lejárat

[juhasz]

renld magadhoz a feladatot, ha ezzel akarsz foglalkozni!

[barnabas-nagy]

Kész a feladat.

[barnabas-nagy]

Módosított .info:
http://dl.dropbox.com/u/5781492/shrtr.info

Módosított .module:
http://dl.dropbox.com/u/5781492/shrtr.module

A date module követelmény, azon belül a Date API és a Date Popup (<=mert ez jól néz ki.)

[juhasz]

Amit megcsináltál, az működik, de még nincs kész.

1. Eddig volt egy választólista, ahol ki lehet választani, hogy még mennyi ideig érvényes, ez a módosítás ezt cseréli le egy dátumválasztóra. Az elvárt működés az lenne, hogy legyen lehetőség mindkét módszerre.
2. Ne legyen függőség a date és a date popoup. Lehet, hogy valaki nem is akarja használni a lejárati idő beállítást, akkor meg minek telepítsen két extra modult? Helyette használjuk a rendes date field-et, ami "nem olyan jól néz ki", és vizsgáljuk meg, hogy van-e date popup modul, ha van, akkor használja azt.

[barnabas-nagy]

Date field létezik a date modul nélkül is?
a sima date-modulnak mindenképpen függőségnek kell lennie, nem?
Date mezőnk alapból van?

1. április 19. 12:51 Márton Juhász írta, <
   reply@reply.github.com

   :

Amit megcsináltál, az működik, de még nincs kész.

1. Eddig volt egy választólista, ahol ki lehet választani, hogy még mennyi
   ideig érvényes, ez a módosítás ezt cseréli le egy dátumválasztóra. Az
   elvárt működés az lenne, hogy legyen lehetőség mindkét módszerre.
2. Ne legyen függőség a date és a date popoup. Lehet, hogy valaki nem is
   akarja használni a lejárati idő beállítást, akkor meg minek telepítsen két
   extra modult? Helyette használjuk a rendes date field-et, ami "nem olyan
   jól néz ki", és vizsgáljuk meg, hogy van-e date popup modul, ha van, akkor
   használja azt.

---

Reply to this email directly or view it on GitHub:
#7 (comment)

[juhasz]

http://api.drupal.org/api/drupal/developer%21topics%21forms_api_reference.html/7#date

[barnabas-nagy]

A date fieldnek nem lehet üres értéket adni, azaz alapértelmezetten mindig az aktuális dátum látszik. Emiatt nem lehet megoldani, hogy csak akkor vegye figyelembe a dátum szerinti lejárati időt, ha be van állítva és alapértelmezetten a hátralévő idő legyen.
emiatt be kellett vezetnem egy választási lehetőséget (radios field), amivel először ki kell választani, hogy melyik szerint.
Gondolom ez nem gond, most jobb ötletem nincs.
(viszont e szerint js-el el lehet tüntetni a nem használt beviteli módot)

[barnabas-nagy]

A módosított megoldás:
.module:
http://dl.dropbox.com/u/5781492/shrtr.module
.info:
http://dl.dropbox.com/u/5781492/shrtr.info