[Risolto] Bug(?): taggare documenti nella Content Library

Domande sulle funzionalità di backend: amministrazione utenti e corsi, certificati, iscrizioni, report, ...
Amigamerlin
FormaLms User
Posts: 46
Joined: Sun Apr 22, 2012 10:58 am

Re: [Risolto] Bug(?): taggare documenti nella Content Librar

Post by Amigamerlin »

alberto wrote: ... Purtroppo con queste premesse è normale che escano ancora piccoli bug, e sulle funzioni più nuove come appunto la content library è ovviamente ancora più facile.
Comunque sono sicuro che con l'apporto di tutti, condividendo fix e modifiche, riusciremo a renderla una piattaforma ancora migliore....grazie per il grandissimo supporto che stai dando al progetto! :)
Ciao alberto, non per polemizzare, lungi da me ... ma non definirei gli stessi piccoli bug.
Comprendo e condivido al 100% la metafora di Robykyrk
Pensavo di ridipingere casa ed invece tocca restaurarla.
La riscrittura a mio avviso non può inficiare di fatto l'utilizzo della piattaforma. Molti come me hanno scelto docebo perchè opensource e con una community attiva che segnalava prontamente bug e problematiche. Il cambio di revisione ha visto, invece del potenziamento di questi due strumenti, la loro totale chiusura. Quindi persone che hanno creduto nel progetto Docebo open si trovano oggi a gestire e sanare problematiche strutturali di una certa importanza che inficiano funzionalità di base della piattaforma stessa con l'aggravante dell'atteggiamento pilatesco di chi il progetto open lo ha messo in piedi. E' come se ci avessero lasciato il cerino acceso in mano dicendoci "gestitevelo da soli". Questo ovviamente lascia l'amaro in bocca.
Mi permetto di parlare in questo modo con cognizione di causa e non per arroganza o presunzione (cercate il mio nick in google) .

Alberto, perdonami lo sfogo, ma per chi come me in questi progetti open ci crede è come prendere un sonoro pugno tra i denti.
Grazie a tutti per l'attenzione.
User avatar
robykirk
FormaLms User
Posts: 140
Joined: Mon Mar 12, 2012 10:06 pm

Re: [Risolto] Bug(?): taggare documenti nella Content Librar

Post by robykirk »

marcello wrote:Robykirk, in effetti la seconda patch impedisce la ricerca con i tag.
Mi sono anche accorto di un altro problemino, che si pensava risolto con la prima patch: come docente del corso non è possibile inserire i tag con la funzione Categorizza. Anzi, se c'erano già dei tag assegnati dall'amministratore, il tentativo del docente di inserirne altri li cancella tutti.
Ciao
Ok, adesso la prima patch dovrebbe essere completa.
In aggiunta al problema del docente, mi sono reso conto che esisteva un altro piccolo bug che, in caso di cancellazione di un tag manteneva un tag vuoto.
Esempio:
1) inserivo il tag TAG1
2) Cancellavo il TAG1
3) Inserivo il TAG2 ed appariva come , TAG2

Per rendere la modifica più chiara, cancello l'allegato precedente e ripropongo in questo post tutta la modifica apportata.

1) doceboLms\lib\lib.kbres.php

Modificato il seguente codice (riga 170 circa):

Code: Select all

			require_once(_base_.'/lib/lib.json.php');
			$json = new Services_JSON();
			$tags_arr = $json->decode($json_tags);
con il codice:

Code: Select all

	//INIZIO MODIFICA ROBYKIRK
      /*
      require_once(_base_.'/lib/lib.json.php');
      $json = new Services_JSON();
			$tags_arr = $json->decode($json_tags);
      */
      $json_tags = str_replace("[", "", $json_tags);
      $json_tags = str_replace("]", "", $json_tags);
      $json_tags = str_replace('"', "", $json_tags);
      $json_tags = str_replace("\\", "", $json_tags);
      $tags_arr = explode(",", $json_tags);
      //FINE MODIFICA ROBYKIRK

Modificato il seguente codice (riga 577 circa):

Code: Select all

if (!in_array($tag_name, $current_tags)) {
    $to_add[]=$tag_name;
}
else {
    $tags_not_to_rem[]=$tag_name;
}
con il codice:

Code: Select all

if (!in_array($tag_name, $current_tags)) {
      // INIZIO MODIFICA ROBYKIRK  
        if ($tag_name == "") {
          $tags_not_to_rem[]=$tag_name;
	}else {
          $to_add[]=$tag_name;
	}
}else {
	$tags_not_to_rem[]=$tag_name;
}
      // FINE MODIFICA ROBYKIRK

2) doceboLms\modules\organization\orgcategorize.php

Modificato il seguente codice (riga 119 circa):

Code: Select all

			$json = new Services_JSON();
			$tags_arr = $json->decode($json_tags);
con il codice:

Code: Select all

	//INIZIO MODIFICA ROBYKIRK
      /*
      $json = new Services_JSON();
      $tags_arr = $json->decode($json_tags);
      */
      $json_tags = str_replace("[", "", $json_tags);
      $json_tags = str_replace("]", "", $json_tags);
      $json_tags = str_replace('"', "", $json_tags);
      $json_tags = str_replace("\\", "", $json_tags);
      $tags_arr = explode(",", $json_tags);
      //FINE MODIFICA ROBYKIRK

Come al solito, provatelo a fatemi sapere se vi funziona correttamente.
Allego il nuovo file.

Ricordate di mantenere una copia del file originale prima di apportare le modifiche

Ciao
Attachments
Fix TAG Content library.zip
(10.45 KiB) Downloaded 207 times
Roby Kirk
User avatar
robykirk
FormaLms User
Posts: 140
Joined: Mon Mar 12, 2012 10:06 pm

Re: [Risolto] Bug(?): taggare documenti nella Content Librar

Post by robykirk »

marcello wrote:Robykirk, in effetti la seconda patch impedisce la ricerca con i tag.
Dovrei aver risolto anche il problema della seconda patch.
Non utilizzate quella precedentemente allegata.
Utilizzate i soli allegati di questo post (patch2) e quello precedentemente postato in data odierna (patch1)


Ho modificato il file \doceboLms\admin\models\KbAlms.php intorno alla riga 913, sostituendo il seguente codice:

Code: Select all

$where.="( " .
	"(kr.r_env = 'course_lo' AND kr.r_env_parent_id IN (" . implode(',', $courses_arr) . ")) OR " .
	"(kr.r_env = 'communication' AND kr.r_env_parent_id IN (" . implode(',', $comm_arr) . ")) OR " .
	"(kr.r_env = 'games' AND kr.r_env_parent_id IN (" . implode(',', $games_arr) . ")) " .
")";
con il codice:

Code: Select all

$where.="( " .
	"(kr.r_env = 'course_lo' AND kr.r_env_parent_id IN (" . implode(',', $courses_arr) . ")) OR " .
	"(kr.r_env = 'communication' AND kr.r_env_parent_id IN (" . implode(',', $comm_arr) . ")) OR " .
	// INIZIO MODIFICA ROBYKIRK
        //"(kr.r_env = 'games' AND kr.r_env_parent_id IN (" . implode(',', $games_arr) . ")) " .
        "(kr.r_env = 'games' AND kr.r_env_parent_id IN (" . implode(',', $games_arr) . ")) OR " .
        "(kr.force_visible='1') " .
        // FINE MODIFICA ROBYKIRK
")";
Ho fatto diversi test e mi sembra che le 2 pach funzionino correttamente insieme.
Fate dei test anche voi e fateci sapere.

Allego il nuovo file cancellando quello errato dal post precedente.

Ricordate di mantenere una copia del file originale prima di apportare le modifiche


Ciao
Attachments
Fix Visualizzazione LO in Content library.zip
(7.98 KiB) Downloaded 215 times
Roby Kirk
marcello
FormaLms User
Posts: 51
Joined: Tue Apr 10, 2012 9:41 am

Re: [Risolto] Bug(?): taggare documenti nella Content Librar

Post by marcello »

Robykirk,
stavolta sembra davvero funzionare tutto e bene.
Docebo adesso può essere utilizzato più efficacemente da chi opera professionalmente nell'e-learning.
Grazie ancora per aver messo le tue competenze a servizio di questo progetto con tanto impegno e pazienza.
User avatar
Luisiana
FormaLms User
Posts: 50
Joined: Thu Mar 01, 2012 3:05 pm
Contact:

Re: [Risolto] Bug(?): taggare documenti nella Content Librar

Post by Luisiana »

Il Bug segnalato è stato fixato su Forma. Grazie a tutti per la collaborazione :)
Post Reply