anmerkung: contenido-cvs-2005-12-23.tar.tar

Beitrag von **HerrB** » So 25. Dez 2005, 21:39

Wird eine Historie übernommen, geht die Beschreibung verloren.

Korrektur:

Index: contenido/includes/include.mod_history.php
===================================================================
RCS file: /Development/CVSRepository/contenido_sources/contenido/includes/include.mod_history.php,v
retrieving revision 1.2
diff -u -r1.2 include.mod_history.php
--- contenido/includes/include.mod_history.php	11 Dec 2005 18:48:24 -0000	1.2
+++ contenido/includes/include.mod_history.php	25 Dec 2005 20:35:59 -0000
@@ -42,7 +42,7 @@
         	$mod = new cApiModuleHistory;
         	$mod->loadByPrimaryKey($idmodhistory);
         	
-        	$idmod = modEditModule($idmod, $mod->get("name"), addslashes($mod->get("descr")), addslashes($mod->get("input")), addslashes($mod->get("output")), addslashes($mod->get("template")), addslashes($mod->get("type")));      
+        	$idmod = modEditModule($idmod, $mod->get("name"), addslashes($mod->get("description")), addslashes($mod->get("input")), addslashes($mod->get("output")), addslashes($mod->get("template")), addslashes($mod->get("type")));      
         }
         
         if ($action == "mod_history_clear")

Manuell:
In contenido/includes/include.mod_history.php im Bereich if ($action == "mod_history_takeover") $mod->get("descr") durch $mod->get("description") ersetzen.

Gruß
HerrB

Beitrag von **HerrB** » Sa 31. Dez 2005, 16:09

In class.htmlelements.php findet sich bei cHTMLDiv und cHTMLSpan eine ähnliche Initialisierungsroutine:

Code: Alles auswählen

function cHTMLDIV($content = NULL)
	{
		cHTML :: cHTML();
		$this->setContentlessTag(false);
		$this->_tag = "div";
	}

Innerhalb der Routinen fehlt die Übernahme des $content, d.h. $this->_setContent($content); so dass der Inhalt (trotz vorgesehenem Parameter) nicht schon gleich bei der Definition angegeben werden kann.

Gruß
HerrB

Beitrag von **emergence** » Mo 2. Jan 2006, 15:25

ich häng da was ran, was bei mir ein problem beim testen der conEditFirstTime und conEditArt verursacht hat...

ist etwas trivial

db tabelle con_actions bei wert für con_saveart feld code ist der code erstens teilweise doppelt vorhanden (aufgrund dessen das einmal $idart nicht existiert) und zweitens wird beim setzen des flags für die code generierung die variable $idcat neu gesetzt...

hier eine korrigierte version

Code: Alles auswählen

if (!isset($idtpl))
{
  $idtpl = false;
}

if (!isset($artspec))
{
  $artspec = "";
}

if (!isset($online))
{
  $online = false;
}

if (isset($title)) 
{
	
	if (1 == $tmp_firstedit) 
	{
		$idart = conEditFirstTime($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort);
	} else {
		conEditArt($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort);
	}

    $tmp_notification = $notification->returnNotification("info", i18n("Changes saved"));

    if ( !isset($idartlang) )
    {
        $sql = "SELECT idartlang FROM ".$cfg["tab"]["art_lang"]." WHERE idart = $idart AND idlang = $lang";
        $db->query($sql);
        $db->next_record();
        $idartlang = $db->f("idartlang");
    }

    if ( is_array($idcatnew) )
    {
        if ( in_array($idcat, $idcatnew) )
        {

            $sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '".$idcat."' AND idart = '".$idart."'";

            $db->query($sql);
            $db->next_record();

            $tmp_idcatart = $db->f("idcatart");

            if ( $is_start == 1 )
            {
                conMakeStart($tmp_idcatart, $is_start);
            }

            if (!isset($is_start))
            {
                if ($cfg["is_start_compatible"] == true)
                {

                    $sql = "SELECT * FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$idcat' AND is_start = '1' ";
                    $db->query($sql);
                    if ( !$db->next_record() )
                    {
                        conMakeStart($tmp_idcatart, 0);
                    }

                } else {

                    $sql = "SELECT * FROM ".$cfg["tab"]["cat_lang"]." WHERE idcat = '$idcat' AND idlang = '$lang' AND startidartlang != '0' ";
                    $db->query($sql);
                    if ( $db->next_record() )
                    {
                        $tmp_startidartlang = $db->f('startidartlang');
                        if ($idartlang == $tmp_startidartlang)
                        {
                            conMakeStart($tmp_idcatart, 0);
                        }

                    } else {
                        conMakeStart($tmp_idcatart, 0);
                    }

                }

            }
        }
    }

    if ( is_array($idcatnew) )
    {

        foreach ( $idcatnew as $tmp_idcat )
        {

            $sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$tmp_idcat' AND idart = '$idart'";

            $db->query($sql);
            $db->next_record();

            conSetCodeFlag( $db->f("idcatart") );

        }
    }

}

Beitrag von **emergence** » Di 3. Jan 2006, 13:34

noch was zum vorigem posting

Code: Alles auswählen

    if ( is_array($idcatnew) )
    {

        foreach ( $idcatnew as $tmp_idcat )
        {

            $sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$tmp_idcat' AND idart = '$idart'";

            $db->query($sql);
            $db->next_record();

            conSetCodeFlag( $db->f("idcatart") );

        }
    }

hätte da ein

Code: Alles auswählen

    conGenerateCodeForArtInAllCategories($idart);

nicht gereicht ?

Beitrag von **emergence** » Di 3. Jan 2006, 13:41

wieder mal durch zufall draufgenagelt...

contenido\classes\class.xml.php

Code: Alles auswählen

    function load($filename) {

        if (file_exists($filename)) {

sollte aber

Code: Alles auswählen

    function load($filename) {

        if (file_exists($filename) && !is_dir($filename)) {

sein...

Beitrag von **HerrB** » Di 3. Jan 2006, 18:36

Ähm, auf welche Datei bezieht sich das Posting: http://www.contenido.org/forum/viewtopi ... 6666#66666?

Gruß
HerrB

Beitrag von **emergence** » Di 3. Jan 2006, 18:40

das ist keine datei
siehe db -> tabelle con_actions -> bei wert für con_saveart -> feld code

Beitrag von **HerrB** » Di 3. Jan 2006, 18:56

Ah

Gruß
HerrB

Beitrag von **emergence** » Fr 6. Jan 2006, 11:31

hmm...

ich hab jetzt mal versucht die classes/class.ui.php im frontend zu verwenden...
ähm das geht nicht wirklich gut, da man dauernd auf eine login aufforderung kommt...

im code findet sich 2x

Code: Alles auswählen

$this->formvars["contenido"] = $sess->id;

sollte auf

Code: Alles auswählen

$this->formvars[$sess->name] = $sess->id;

geändert werden...

Beitrag von **HerrB** » Do 8. Jun 2006, 17:29

Bis auf http://www.contenido.org/forum/viewtopi ... 6666#66666
done.

Gruß
HerrB

Beitrag von **emergence** » Mi 14. Jun 2006, 08:22

ich hab den code nochmals durchgesehen... und mit der zweiten anmerkung ergänzt...

sollte anstandslos laufen... (ist kürzer und verständlicher)

con_saveart orginal:

Code: Alles auswählen

if (!isset($idtpl))
{
  $idtpl = false;
}

if (!isset($artspec))
{
  $artspec = "";
}

if (!isset($online))
{
  $online = false;
}

if (isset($title)) 
{
	
	if (1 == $tmp_firstedit) 
	{
	
		$idart = conEditFirstTime($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort);
		$tmp_notification = $notification->returnNotification("info", i18n("Changes saved"));
		
		if ( !isset($idartlang) ) 
		{
			$sql = "SELECT idartlang FROM ".$cfg["tab"]["art_lang"]." WHERE idart = $idart AND idlang = $lang";
			$db->query($sql);
			$db->next_record();
			$idartlang = $db->f("idartlang");
		}
		
		if ( in_array($idcat, $idcatnew) ) 
		{
		
			$sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '".$idcat."' AND idart = '".$idart."'";
			
			$db->query($sql);
			$db->next_record();
			
			$tmp_idcatart = $db->f("idcatart");
			
			if ( $is_start == 1 ) 
			{
				conMakeStart($tmp_idcatart, $is_start);
			}
			
			if (!isset($is_start))
			{
				if ($cfg["is_start_compatible"] == true)
				{
				
					$sql = "SELECT * FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$idcat' AND is_start = '1' ";
					$db->query($sql);
					if ( $db->next_record() ) 
					{
						; 
					}else
					{
						conMakeStart($tmp_idcatart, 0);
					}
					
				}else
				{
				
					$sql = "SELECT * FROM ".$cfg["tab"]["cat_lang"]." WHERE idcat = '$idcat' AND idlang = '$lang' AND startidartlang != '0' ";
					$db->query($sql);
					if ( $db->next_record() ) 
					{
						$tmp_startidartlang = $db->f('startidartlang');
						if ($idartlang == $tmp_startidartlang)
						{
							conMakeStart($tmp_idcatart, 0);
						}else
						{
							; # do nothing
						}
						
					}else
					{
						conMakeStart($tmp_idcatart, 0);
					}
					
				}
			}
			
		}
	
		if ( is_array($idcatnew) ) 
		{
		
			foreach ( $idcatnew as $idcat ) 
			{
			
				$sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = $idcat AND idart = $idart";
				
				$db->query($sql);
				$db->next_record();
				
				conSetCodeFlag( $db->f("idcatart") );
			
			}
		}
	
	}	
	else 
	{
	
		conEditArt($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort);
		
		$tmp_notification = $notification->returnNotification("info", i18n("Changes saved"));
		
		if ( !isset($idartlang) ) 
		{
			$sql = "SELECT idartlang FROM ".$cfg["tab"]["art_lang"]." WHERE idart = $idart AND idlang = $lang";
			$db->query($sql);
			$db->next_record();
			$idartlang = $db->f("idartlang");
		}
		
		if ( is_array($idcatnew) ) 
		{
			if ( in_array($idcat, $idcatnew) ) 
			{
			
				$sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '".$idcat."' AND idart = '".$idart."'";
				
				$db->query($sql);
				$db->next_record();
				
				$tmp_idcatart = $db->f("idcatart");
				
				if ( $is_start == 1 ) 
				{
					conMakeStart($tmp_idcatart, $is_start);
				}
				
				if (!isset($is_start))
				{
					if ($cfg["is_start_compatible"] == true)
					{
					
						$sql = "SELECT * FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$idcat' AND is_start = '1' ";
						$db->query($sql);
						if ( $db->next_record() ) 
						{
							; 
						}else
						{
							conMakeStart($tmp_idcatart, 0);
						}
						
					}else
					{
					
						$sql = "SELECT * FROM ".$cfg["tab"]["cat_lang"]." WHERE idcat = '$idcat' AND idlang = '$lang' AND startidartlang != '0' ";
						$db->query($sql);
						if ( $db->next_record() ) 
						{
							$tmp_startidartlang = $db->f('startidartlang');
							if ($idartlang == $tmp_startidartlang)
							{
								conMakeStart($tmp_idcatart, 0);
							}else
							{
								; # do nothing
							}
							
						}else
						{
							conMakeStart($tmp_idcatart, 0);
						}
						
					}
					
				}
			}
		}
		
		if ( is_array($idcatnew) ) 
		{
		
			foreach ( $idcatnew as $idcat ) 
			{
			
				$sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = $idcat AND idart = $idart";
				
				$db->query($sql);
				$db->next_record();
				
				conSetCodeFlag( $db->f("idcatart") );
				
			}
		}
	}
}

con_saveart neu:

Code: Alles auswählen

if (!isset($idtpl)) 
{ 
  $idtpl = false; 
} 

if (!isset($artspec)) 
{ 
  $artspec = ""; 
} 

if (!isset($online)) 
{ 
  $online = false; 
} 

if (isset($title)) 
{ 
    
   if (1 == $tmp_firstedit) 
   { 
      $idart = conEditFirstTime($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort); 
   } else { 
      conEditArt($idcat, $idcatnew, $idart, $is_start, $idtpl, $idartlang, $lang, $title, $summary, $artspec, $created, $lastmodified, $author, $online, $datestart, $dateend, $artsort); 
   } 

    $tmp_notification = $notification->returnNotification("info", i18n("Changes saved")); 

    if ( !isset($idartlang) ) 
    { 
        $sql = "SELECT idartlang FROM ".$cfg["tab"]["art_lang"]." WHERE idart = $idart AND idlang = $lang"; 
        $db->query($sql); 
        $db->next_record(); 
        $idartlang = $db->f("idartlang"); 
    } 

    if ( is_array($idcatnew) ) 
    { 
        if ( in_array($idcat, $idcatnew) ) 
        { 

            $sql = "SELECT idcatart FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '".$idcat."' AND idart = '".$idart."'"; 

            $db->query($sql); 
            $db->next_record(); 

            $tmp_idcatart = $db->f("idcatart"); 

            if ( $is_start == 1 ) 
            { 
                conMakeStart($tmp_idcatart, $is_start); 
            } 

            if (!isset($is_start)) 
            { 
                if ($cfg["is_start_compatible"] == true) 
                { 

                    $sql = "SELECT * FROM ".$cfg["tab"]["cat_art"]." WHERE idcat = '$idcat' AND is_start = '1' "; 
                    $db->query($sql); 
                    if ( !$db->next_record() ) 
                    { 
                        conMakeStart($tmp_idcatart, 0); 
                    } 

                } else { 

                    $sql = "SELECT * FROM ".$cfg["tab"]["cat_lang"]." WHERE idcat = '$idcat' AND idlang = '$lang' AND startidartlang != '0' "; 
                    $db->query($sql); 
                    if ( $db->next_record() ) 
                    { 
                        $tmp_startidartlang = $db->f('startidartlang'); 
                        if ($idartlang == $tmp_startidartlang) 
                        { 
                            conMakeStart($tmp_idcatart, 0); 
                        } 

                    } else { 
                        conMakeStart($tmp_idcatart, 0); 
                    } 

                } 

            } 
        } 
    } 

    conGenerateCodeForArtInAllCategories($idart); 

}

Beitrag von **emergence** » Mi 9. Aug 2006, 13:06

geschlossen, da es in dieser form nicht mehr benötigt wird...
-> http://contenido.org/forum/viewtopic.php?p=84879#84879