Tekstin lainaaminen toisesta dokumentista #400

Open dezhidki opened 9 years ago

In GitLab by @Smibu on Aug 6, 2015, 22:17

Ominaisuudet

[x] Yksittäisen kappaleen lainaus
[x] Alueen lainaus
[x] Lainatun alueen hintiksi voisi tulla (tai johonkin muualle pienellä) tieto että mistä dokumentista lainattu - TK
[x] Lainatusta tekstistä tapa päästä alkuperäiseen. Käyttötapaus; dokumentti Demo2 on kastattu Ohj1-monisteen tehtävistä ottamalla siihen 8 tehtävää tuolta. Joku haluaa lukea tehtävään 3 liittyvää teoriaa ja haluaa päästä monisteessa oikeaan kohtaan heti. - TK
[ ] Onko käänteinen mahdollista? Eli voisiko Ohj1-monisteeseen saada jotenkin tiedon siitä, mitä kohtia monisteeseen Demo 2 on lainattu. Vähän kuten selaimen Findissa jotta voisi hyppiä lainauksesta toiseen.
[ ] alueille käyttöoikeudet, oletus=owner
[x] varmaan myös muokkaamisen takia voisi haluta hypätä alkuperäiseen dokuun kyseisein lainauksen kohdalle - TK
[x] mitä tapahtuu rekursiivisessa lainaamisessa: eli että lainaan alueen dokusta A dokuun B ja merkitsen sen ympärille isomman alueen Ba ja sitten lainaan Ba:n dokuun C
[ ] jos lainattu paragraph tuhotaan, niin voisiko lainattuun tulla kuitenkin sen viimeisin versio (ehkä jollakin merkillä merkittynä että poistettu, ainakin edit-tilassa, ei välttämättä student_viewssä)
[x] Popup-ikkuna lainauksen yksityiskohdista
[ ] Lainatut kappaleet näkymään sisällysluettelossa
[ ] Dokumentin asetuksiin optio tallennetaanko pluginien vastaukset tähän dokumenttiin vai sinne mistä ne on lainattu

Bugit

[x] Kun tallentaa kappaleen/alueen, joka sisältää lainauksia, kappaleet eivät välttämättä näy oikein selaimessa ennen kuin painaa F5.
[x] Jotta lainattuun pluginkappaleeseen voi tallentaa vastauksen, käyttäjällä pitää olla lukuoikeus alkuperäiseen dokuun, jossa pluginkappale on. Luulisin, että näin ei pitäisi tarvita olla (?). Eli saveAnswer-reitissä täytyy tehdä jokin lisätarkistus ennen kuin lätkäistään status 400. Voinee hoitaa interceptorin avulla lähettämällä lainaavan dokun id:n ja kappaleen id:n, ja reitti sitten tarkistaa, että lainaus vastaa pluginia.
[x] Editoiva käyttäjä pystyy lainailemaan kappaleita mistä tahansa dokumentista, vaikka pitäisi rajoittaa vain niihin, joihin editoijalla on lukuoikeus.
[ ] Lainaavaan kappaleeseen voi kirjoittaa ja tallentaa muutakin sisältötekstiä, mutta sillä ei ole mitään vaikutusta mihinkään, koska vain lainausattribuuteilla on väliä. Tässä ei liene varsinaista korjattavaa muuta kuin ehkä jokin varoitus käyttäjälle.
[ ] Lukumerkinnät lainatun alueen osalta ei toimi ihan järkevästi: jos merkkaa yhden alueesta luetuksi, niin F5 jälkeen koko alue näkyy luettuna. Johtuu siitä, kun alueen kappaleilla on HTML:ssä id:nä lainaavan kappaleen id, joka on alueen sisällä sama.
[ ] Jos teen alueen laittamalla siihen alku/loppumerkin ja sitten valitsen uuden alueen, joka sisältää vain loppumerkin ja vähän muuta, mutta ei alkumerkkiä ja yritän sitten siirtää loppumerkiä cut/pastella toiseen kohtaan, niin herjaa vaikka alkumerkki on olemassa. Näihin molempiin voisi olla, että ennen herjaamista katsotaan koko doku ja jos merkintä poistuu, niin katsotaan jääkö dokuun orpona vastakpl. Tämä voi tosin olla aikavaatimukseltaan iso?
[ ] Luokallinen alue ei tule kokonaan divin sisään vaan pelkkä ensimmäinen kappale.
[x] HTML:n päivityksen refaktorointi

In GitLab by @Smibu on Aug 7, 2015, 21:01

Nyt pystyy kappaleita ja alueita alustavasti lainailemaan. Kirjoittelin tunnettuja bugeja tuonne ylös.

Plussaa: preview tajuaa lainaukset ja ärähtää, jos aluemerkinnät on jotenkin pielessä (puuttuva alueen lopetus jne...) tai jos lainattavaa dokua/kappaletta ei löydy.

Tuolta siis näkee lainaus- ja alueattribuutit: https://tim.it.jyu.fi/view/tim/TIMin%20kehitys/Document%20data%20model

Tuossa HTML:n päivityksessä tallennuksen yhteydessä kannattaa varmaankin ruveta käyttämään samaa koodia, jolla tuo HTML tuodaan ensimmäisen kerran selaimelle. Sitä sitten vaan kutsuttaisiin tietyllä aluemäärityksellä, ja JS vaan suoraan länttäisi palautetun HTML:n dokuun. Myös $compilea pitänee kutsua pluginien varalta. Tällä hetkellä se menee niin, että tallennuksessa palautetaan JSONia, ja sitä sitten parsitaan ja päivitellään HTML:ää sen mukaan. EDIT: Tämä on nyt tehty. @vesal @tomijkarppinen @juvalkee

In GitLab by @Smibu on Aug 11, 2015, 18:37

@vesal @juvalkee @tomijkarppinen Tein äsken tuon HTML-päivitys-parannuksen, niin nyt myös lukumerkinnät tulevat heti varsinaisen dokun mukana ilman ajaxia.

Lukumerkinnät näkyy nyt previewissäkin, mutta ne varmaankin kannattaa sieltä piilottaa.

In GitLab by @Smibu on Aug 11, 2015, 19:04

@Smibu @juvalkee @tomijkarppinen Tuossahan tulee kohta jollekin mergeämistä kun mulla tulee aika moneen paikkaan muutoksia sen lazy-plugins takia. Ja Juusolla pitäisi kai olla että sislu tulee html:änä?

Nopeuttiko tuo yhtään että merkinnät on html:ää? – Vesa Lappalainen

In GitLab by @Smibu on Aug 11, 2015, 20:14

@vesal @juvalkee @tomijkarppinen En mitannut, mutta en keksi miten se voisi ainakaan hitaampi olla kuin entinen tapa. Toki palvelin joutuu lähettämään vähän enemmän tavaraa, mutta ainakaan mobiilit ei joudu JS:llä rassaamaan yhtä paljon.

Noteillekin kannattaa varmaan tehdä samalla tavalla.

In GitLab by @Smibu on Aug 12, 2015, 09:49

@Smibu @juvalkee @tomijkarppinen Saatko Mika mergettyät ämän päähaaraan, niin voin mitata onko tuolla kuinka paljon vaikutusta. Mikäs oli sen Juuson sisällysluettelon tilanne, joko senkin saisi päähaaraan? Onko se notes vaikea tehdä? Vai vaatiiko tämä muutos myös uuden data-modelin tuotantoon ottamisen? Onko sillekään esteitä? – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 10:06

@vesal @Smibu @tomijkarppinen Sisällysluettelo tehdään tällä hetkellä javascriptissä käymällä läpi otsikot. Ei siis vielä pythonin puolella. – Juuso Valkeejärvi

In GitLab by @Smibu on Aug 12, 2015, 10:07

@juvalkee Mutta olikos sulla siihen tekeillä se että se tehdään Pythonin puolella? Sehän tarvitaan siihen paloinakin näyttämiseen. Ei nyt tarvitse prioriteettejä vaihtaa, kunhan pitää tuon mielessä :-) – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 11:47

@vesal Muutin tuota niin, että tehdään osa pythonin puolella ja osa javascriptissä. /index/ reitti palauttaa listan tupleja muodossa (otsikon_id, otsikon_teksti, otsikon_taso). Tuo lista käydään läpi javascriptissä ja luodaan linkit. – Juuso Valkeejärvi

In GitLab by @Smibu on Aug 12, 2015, 11:55

@juvalkee Onkos tuossa joku syy, miksei tuota voisi tehdä kokonaan valmiiksi pythonin puolella, koska tuossahan ei ole mitään dynaamista? – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 13:15

@vesal Pitäis tuon onnistua myös kokonaan pythonin puolella. Jonkun verran siinä käytetään angularia tällä hetkellä, esim noiden expand/collapse juttujen kanssa. – Juuso Valkeejärvi

In GitLab by @Smibu on Aug 12, 2015, 13:16

@juvalkee Onkos niissä jotakin mitä ei voisi tehdä ihan perusjavascriptillä. Pari pientä yksittäistä Angularia ei haittaa, mutta jos on joka rivillä jotakin mihin liittyy dynaamistuutta, niin se käy kalliiksi.

Miten tuo on tuolla tehty:

http://kurssit.it.jyu.fi/ITKP102/monistecs/html/ – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 13:36

@vesal Joo hoituu tuo ihan perusjavascriptillä ja css:llä. – Juuso Valkeejärvi

In GitLab by @Smibu on Aug 12, 2015, 13:47

@juvalkee Katoitko tuota vanhaan Ohj1 monistetta, eikös siinä ole melkein valmis malli? Sitten nuo on erikseen, että haluaako sislun sammuvan kun sitä klikkaa niin että hyppää dokuun. Mobiileissa ihan ok tuo ominaisuus. Josksu se toimi tässä TIMin versiossa, ne muista toimiiko eneää. – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 14:15

@vesal Joo katoin. Sieltä saa melko suoraan ne koodit. – Juuso Valkeejärvi

In GitLab by @Smibu on Aug 12, 2015, 14:19

Vai vaatiiko tämä muutos myös uuden data-modelin tuotantoon ottamisen?

Joo siis tuo DataModel pitäisi ottaa kokonaan käyttöön, sieltä on aika hankala napata yksittäisiä osia, kun muutoksia on tehty jo aika paljon.

Onko sillekään esteitä?

Ei oikeastaan; kyllähän tämä DataModel jo sinänsä toimii.

Onko se notes vaikea tehdä?

Ei pitäisi; ihan samaan tyyliin onnistuu kuin lukumerkinnät.

Eli mun tämän päivän homma voisi olla kommenttien lähetys dokun mukana ja sitten mergetän masterin muutokset DataModeliin. Ehdin katsoa noita tässä illempana.

@vesal @juvalkee @tomijkarppinen

In GitLab by @Smibu on Aug 12, 2015, 14:32

@Smibu Entä sitten ohjeet/ start scriptit dumbon käynistämiseksi, sehän tarvitaan ainakin että saa lokaaleissa koneissa tuon uuden masterin käyntiin. Jääkös ephe vielä sitten johonkin käyttöön? Pillkomaan niitä uusi MD dokuja pieniksi tms? – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 17:23

@vesal Dumbon ja Ephen käynnistys on ihan automaattista, eli launch.py hoitaa sen.

Epheä tarvitaan vain kun importataan vanhat dokut uuteen rakenteeseen.

In GitLab by @Smibu on Aug 12, 2015, 17:26

@Smibu Pitääkös buildata uusia kontteja vai kelpaako joku vanhoista tuolle Dumbollekin? – Vesa Lappalainen

In GitLab by @Smibu on Aug 12, 2015, 17:28

@vesal Ville teki dumbolle oman imagen. build_plugins kääntää senkin.

TIM-JYU / TIM

Tekstin lainaaminen toisesta dokumentista #400

Ominaisuudet

Bugit