Merge branch 'master' into feature-5534/Gehaltsgraph_Valorisierung

application/config/mail.php

@@ -32,3 +32,18 @@ $config['validate'] = false; // If true then the email address will be validated
 
 // If enabled will be logged info about emails in Codeigniter error logs
 $config['enable_debug'] = false;
+
+// default sender
+$config['sancho_mail_default_sender'] = defined('SANCHO_MAIL_DEFAULT_SENDER') ? SANCHO_MAIL_DEFAULT_SENDER : '';
+
+// If to use images for custom mails
+$config['sancho_mail_use_images'] = defined('SANCHO_MAIL_USE_IMAGES') ? SANCHO_MAIL_USE_IMAGES : false;
+
+// image path for sancho mail, relativ to document root
+$config['sancho_mail_img_path'] = defined('SANCHO_MAIL_IMG_PATH') ? SANCHO_MAIL_IMG_PATH : '';
+
+// header image for custom mails
+$config['sancho_mail_header_img'] = defined('SANCHO_MAIL_HEADER_IMG') ? SANCHO_MAIL_HEADER_IMG : '';
+
+// footer image for custom mails
+$config['sancho_mail_footer_img'] = defined('SANCHO_MAIL_FOOTER_IMG') ? SANCHO_MAIL_FOOTER_IMG : '';

application/controllers/jobs/AnrechnungJob.php

@@ -195,10 +195,10 @@ class AnrechnungJob extends JOB_Controller
 			$studiengang_bezeichnung = $this->StudiengangModel->load($studiengang_kz)->retval[0]->stg_bezeichnung;
 
 			// Get STGL mail address
-			$stglMailReceiver_arr = self::_getSTGLMailAddress($studiengang_kz);
+			$stglMailReceiver_arr = $this->_getSTGLMailAddress($studiengang_kz);
 
 			// Get HTML table with new Anrechnungen of that STG plus amount of them
-			list ($anrechnungen_amount, $anrechnungen_table) = self::_getSTGLMailDataTable($studiengang_kz, $anrechnungen);
+			list ($anrechnungen_amount, $anrechnungen_table) = $this->_getSTGLMailDataTable($studiengang_kz, $anrechnungen);
 
 			// Link to Antrag genehmigen dashboard
 			$url =
@@ -514,8 +514,6 @@ html;
 					'vorname' => $stgl->vorname
 				);
 			}
-
-			return $stglMailAdress_arr;
 		}
 		// If not available, get assistance mail address
 		else
@@ -524,12 +522,13 @@ html;
 
 			if (hasData($result))
 			{
-				return array(
-					$result->retval[0]->email,
-					''
+				$stglMailAdress_arr[]= array(
+					'to' => $result->retval[0]->email,
+					'vorname' => ''
 				);
 			}
 		}
+		return $stglMailAdress_arr;
 	}
 
 	// Build HTML table with yesterdays new Anrechnungen of the given STG

application/controllers/jobs/AntragJob.php

@@ -183,8 +183,8 @@ class AntragJob extends JOB_Controller
 				$data,
 				$to,
 				'Anträge - Aktion(en) erforderlich',
-				DEFAULT_SANCHO_HEADER_IMG,
-				DEFAULT_SANCHO_FOOTER_IMG,
+				'',
+				'',
 				'',
 				$cc
 			))

application/controllers/jobs/ReihungstestJob.php

@@ -431,8 +431,8 @@ class ReihungstestJob extends JOB_Controller
 								$mailcontent_data_arr,
 								$applicant->email,
 								'Ihre Anmeldung zum Reihungstest - Reminder / Your registration for the placement test - Reminder',
-								DEFAULT_SANCHO_HEADER_IMG,
-								DEFAULT_SANCHO_FOOTER_IMG,
+								'',
+								'',
 								$from,
 								'',
 								$bcc);

application/core/DB_Model.php

@@ -991,14 +991,17 @@ class DB_Model extends CI_Model
 
 						// Find and replace all the occurrences of the provided encrypted columns
 						// with the postgresql decryption function
-						$query = preg_replace(
-							'/\b' . $encryptedColumn . '\b/',
-							sprintf(
-								self::CRYPT_WHERE_TEMPLATE,
-								$encryptedColumn,
-								$decryptionPassword,
-								$definition[self::CRYPT_CAST]
-							),
+						$query = preg_replace_callback(
+							'/(?<! (as|AS) )\b(\w+\.)?(' . $encryptedColumn . ')\b/',
+							function($matches) use (&$decryptionPassword, &$definition) {
+							    $aliased_column = $matches[2] . $matches[3];
+								return sprintf(
+									self::CRYPT_WHERE_TEMPLATE,
+									$aliased_column,
+									$decryptionPassword,
+									$definition[self::CRYPT_CAST]
+								);
+							},
 							$query
 						);
 					}
@@ -1106,14 +1109,17 @@ class DB_Model extends CI_Model
 						{
 							// Find and replace all the occurrences of the provided encrypted columns
 							// with the postgresql decryption function
-							$where = preg_replace(
-								'/\b' . $encryptedColumn . '\b/',
-								sprintf(
-									self::CRYPT_WHERE_TEMPLATE,
-									$encryptedColumn,
-									$decryptionPassword,
-									$definition[self::CRYPT_CAST]
-								),
+							$where = preg_replace_callback(
+								'/(?<! (as|AS) )\b(\w+\.)?(' . $encryptedColumn . ')\b/',
+								function($matches) use (&$decryptionPassword, &$definition) {
+									$aliased_column = $matches[2] . $matches[3];
+									return sprintf(
+										self::CRYPT_WHERE_TEMPLATE,
+										$aliased_column,
+										$decryptionPassword,
+										$definition[self::CRYPT_CAST]
+									);
+								},
 								$where
 							);
 						}

application/helpers/hlp_sancho_helper.php

@@ -23,9 +23,6 @@ if (! defined('BASEPATH')) exit('No direct script access allowed');
 // Functions needed in the view FHC-Header
 // ------------------------------------------------------------------------
 
-const DEFAULT_SANCHO_HEADER_IMG = 'sancho_header_DEFAULT.jpg';
-const DEFAULT_SANCHO_FOOTER_IMG = 'sancho_footer_DEFAULT.jpg';
-
 /**
  * Send single Mail with Sancho Design and Layout.
  * @param string $vorlage_kurzbz Name of the template for specific mail content.
@@ -38,27 +35,88 @@ const DEFAULT_SANCHO_FOOTER_IMG = 'sancho_footer_DEFAULT.jpg';
  * @param string $bcc Sets BCC of mail.
  * @return void
  */
-function sendSanchoMail($vorlage_kurzbz, $vorlage_data, $to, $subject, $headerImg = DEFAULT_SANCHO_HEADER_IMG, $footerImg = DEFAULT_SANCHO_FOOTER_IMG, $from = null, $cc = null, $bcc = null)
+function sendSanchoMail(
+	$vorlage_kurzbz,
+	$vorlage_data,
+	$to,
+	$subject,
+	$headerImg = '',
+	$footerImg = '',
+	$from = null,
+	$cc = null,
+	$bcc = null
+)
 {
 	$ci =& get_instance();
 	$ci->load->library('email');
 	$ci->load->library('MailLib');
 
-	$sanchoHeader_img = 'skin/images/sancho/'. $headerImg;
-	$sanchoFooter_img = 'skin/images/sancho/'. $footerImg;
+	$sancho_mail_config = $ci->config->item('mail');
+
 
 	if ($from == '')
 	{
-		$from = 'sancho@'.DOMAIN;
+		$from = ((isset($sancho_mail_config['sancho_mail_default_sender'])
+				&& $sancho_mail_config['sancho_mail_default_sender']) 
+			  ? $sancho_mail_config['sancho_mail_default_sender']
+			  : 'noreply')
+			  . '@' . DOMAIN;
 	}
 
 	// Embed sancho header and footer image
 	// reset important to ensure embedding of images when called in a loop
 	$ci->email->clear(true);  // clear vars and attachments
-	$ci->email->attach($sanchoHeader_img);
-	$ci->email->attach($sanchoFooter_img);
-	$cid_header = $ci->email->attachment_cid($sanchoHeader_img); // sets unique content id for embedding
-	$cid_footer = $ci->email->attachment_cid($sanchoFooter_img); // sets unique content id for embedding
+
+	$cid_header = '';
+	$cid_footer = '';
+
+	if (isset($sancho_mail_config['sancho_mail_use_images']) && $sancho_mail_config['sancho_mail_use_images'])
+	{
+		$sanchoHeader_img = '';
+		$sanchoFooter_img = '';
+
+		if (isset($headerImg) && $headerImg != '')
+		{
+			// use provided header image
+			$sanchoHeader_img = $headerImg;
+		}
+		elseif (isset($sancho_mail_config['sancho_mail_header_img']) && $sancho_mail_config['sancho_mail_header_img'])
+		{
+			// use default header image
+			$sanchoHeader_img = $sancho_mail_config['sancho_mail_header_img'];
+		}
+
+		if (isset($footerImg) && $footerImg != '')
+		{
+			// use provided footer image
+			$sanchoFooter_img = $footerImg;
+		}
+		elseif (isset($sancho_mail_config['sancho_mail_footer_img']) && $sancho_mail_config['sancho_mail_footer_img'])
+		{
+			// use default footer image
+			$sanchoFooter_img =  $sancho_mail_config['sancho_mail_footer_img'];
+		}
+
+		// add image file paths
+		if (isset($sancho_mail_config['sancho_mail_img_path']))
+		{
+			if ($sanchoHeader_img != '')
+			{
+				$sanchoHeader_img = $sancho_mail_config['sancho_mail_img_path'].$sanchoHeader_img;
+			}
+
+			if ($sanchoFooter_img != '')
+			{
+				$sanchoFooter_img = $sancho_mail_config['sancho_mail_img_path'].$sanchoFooter_img;
+			}
+		}
+
+		// attach header and footer
+		$ci->email->attach($sanchoHeader_img, 'inline');
+		$ci->email->attach($sanchoFooter_img, 'inline');
+		$cid_header = $ci->email->attachment_cid($sanchoHeader_img); // sets unique content id for embedding
+		$cid_footer = $ci->email->attachment_cid($sanchoFooter_img); // sets unique content id for embedding
+	}
 
 	// Set specific mail content into specific content template
 	$content = _parseMailContent($vorlage_kurzbz, $vorlage_data);
@@ -74,7 +132,18 @@ function sendSanchoMail($vorlage_kurzbz, $vorlage_data, $to, $subject, $headerIm
 	$body = _parseMailContent('Sancho_Mail_Template', $layout);
 
 	// Send mail
-	return $ci->maillib->send($from, $to, $subject, $body, $alias = '', $cc, $bcc, $altMessage = '', $bulk = true, $autogenerated = true);
+	return $ci->maillib->send(
+		$from,
+		$to,
+		$subject,
+		$body,
+		'', // alias
+		$cc,
+		$bcc,
+		'', // altMessage
+		true, // bulk
+		true // autogenerated
+	);
 }
 
 /**

application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php

@@ -33,9 +33,12 @@ class GehaltsbestandteilLib
 		return $this->GehaltsbestandteilModel->getGehaltsbestandteileValorisiert($dienstverhaeltnis_id, $stichtag, $includefuture);
 	}
 
-	public function fetchGehaltsbestandteile($dienstverhaeltnis_id, $stichtag=null, $includefuture=false)
+	public function fetchGehaltsbestandteile($dienstverhaeltnis_id, $stichtag=null, 
+		$includefuture=false, $withvalorisationhistory=true)
 	{
-		return $this->GehaltsbestandteilModel->getGehaltsbestandteile($dienstverhaeltnis_id, $stichtag, $includefuture);
+		return $this->GehaltsbestandteilModel->getGehaltsbestandteile(
+			$dienstverhaeltnis_id, $stichtag, $includefuture, $withvalorisationhistory
+		);
 	}
 
 	public function fetchGehaltsbestandteil($gehaltsbestandteil_id)

application/libraries/vertragsbestandteil/VertragsbestandteilLib.php

@@ -26,6 +26,8 @@ class VertragsbestandteilLib
 {
 	const INCLUDE_FUTURE = true;
 	const DO_NOT_INCLUDE_FUTURE = false;
+	const WITH_VALORISATION_HISTORY = true;
+	const NOT_WITH_VALORISATION_HISTORY = false;
 
 	protected $CI;
 	/** @var Dienstverhaeltnis_model */
@@ -90,10 +92,15 @@ class VertragsbestandteilLib
 		return $dv;
 	}
 
-	public function fetchVertragsbestandteile($dienstverhaeltnis_id, $stichtag=null, $includefuture=false)
+	public function fetchVertragsbestandteile($dienstverhaeltnis_id, $stichtag=null, 
+		$includefuture=false, $withvalorisationhistory=true)
 	{
-		$vbs = $this->VertragsbestandteilModel->getVertragsbestandteile($dienstverhaeltnis_id, $stichtag, $includefuture);
-		$gbs = $this->GehaltsbestandteilLib->fetchGehaltsbestandteile($dienstverhaeltnis_id, $stichtag, $includefuture);
+		$vbs = $this->VertragsbestandteilModel->getVertragsbestandteile(
+			$dienstverhaeltnis_id, $stichtag, $includefuture
+		);
+		$gbs = $this->GehaltsbestandteilLib->fetchGehaltsbestandteile(
+			$dienstverhaeltnis_id, $stichtag, $includefuture, $withvalorisationhistory
+		);
 
 		$gbsByVBid = array();
 		foreach( $gbs as $gb )

application/models/vertragsbestandteil/Gehaltsbestandteil_model.php

@@ -33,41 +33,84 @@ class Gehaltsbestandteil_model extends DB_Model implements IEncryption
         $datestring = $date->format("Y-m-d");
 
 		$qry = "
-        SELECT
-            gehaltsbestandteil_id,
-            gbt.von,
-            gbt.bis,
-            gbt.anmerkung,
-            gbt.dienstverhaeltnis_id,
-            gehaltstyp_kurzbz,
-            valorisierungssperre,
-            gbt.valorisierung,
-            grundbetrag as grund_betrag_decrypted,
-            betrag_valorisiert as betrag_val_decrypted,
-            gt.bezeichnung as gehaltstyp_bezeichnung,
-			vb.vertragsbestandteiltyp_kurzbz,
-			bf.funktion_kurzbz,
-			bf.oe_kurzbz,
-			fkt.beschreibung as fkt_beschreibung,
-			fb.bezeichnung as fb_bezeichnung,
-			org.bezeichnung as org_bezeichnung,
-			freitext.freitexttyp_kurzbz,
-			freitext.titel as freitext_titel
-        FROM hr.tbl_gehaltsbestandteil gbt LEFT JOIN hr.tbl_gehaltstyp gt using(gehaltstyp_kurzbz)
-			LEFT JOIN hr.tbl_vertragsbestandteil vb using(vertragsbestandteil_id)
-			LEFT JOIN hr.tbl_vertragsbestandteil_funktion vbf using(vertragsbestandteil_id)
-			LEFT JOIN public.tbl_benutzerfunktion bf using(benutzerfunktion_id)
-			LEFT JOIN public.tbl_funktion fkt using(funktion_kurzbz)
-			LEFT JOIN public.tbl_fachbereich fb using(fachbereich_kurzbz)
-			LEFT JOIN public.tbl_organisationseinheit org on (bf.oe_kurzbz=org.oe_kurzbz)
-			LEFT JOIN hr.tbl_vertragsbestandteil_freitext freitext on(vb.vertragsbestandteil_id=freitext.vertragsbestandteil_id)
-        WHERE gbt.dienstverhaeltnis_id=? AND
-			(gbt.von<=? and (gbt.bis is null OR gbt.bis>=?))
-        ORDER BY gt.sort
+with gbt as (
+	select 
+	    gb.gehaltsbestandteil_id,
+	    gb.von,
+	    gb.bis,
+	    gb.anmerkung,
+	    gb.dienstverhaeltnis_id,
+	    gb.gehaltstyp_kurzbz,
+	    gb.valorisierungssperre,
+	    gb.valorisierung,
+	    gb.grundbetrag as grund_betrag_decrypted,
+	    coalesce(vh.betrag_valorisiert, gb.grundbetrag) as betrag_val_decrypted,
+	    gb.vertragsbestandteil_id
+	from 
+		hr.tbl_gehaltsbestandteil gb
+	LEFT JOIN 
+		hr.tbl_valorisierung_historie vh ON vh.gehaltsbestandteil_id = gb.gehaltsbestandteil_id AND vh.valorisierungsdatum = (
+	      	SELECT 
+	          	vi.valorisierungsdatum 
+	          FROM 
+	          	hr.tbl_valorisierung_instanz vi
+	          JOIN
+	            hr.tbl_dienstverhaeltnis d ON d.dienstverhaeltnis_id = ? 
+				AND d.oe_kurzbz = vi.oe_kurzbz
+	          WHERE 
+	          	? >= valorisierungsdatum 
+	          ORDER BY 
+	          	valorisierungsdatum DESC 
+	          LIMIT 1
+	        )
+	where
+		dienstverhaeltnis_id = ?
+		and (
+			? BETWEEN COALESCE(von, '1970-01-01'::date) AND COALESCE(bis, '2170-01-01'::date) 
+		)
+)
+select
+    gbt.gehaltsbestandteil_id,
+    gbt.von,
+    gbt.bis,
+    gbt.anmerkung,
+    gbt.dienstverhaeltnis_id,
+    gbt.gehaltstyp_kurzbz,
+    gbt.valorisierungssperre,
+    gbt.valorisierung,
+    gbt.grund_betrag_decrypted,
+    gbt.betrag_val_decrypted,
+	gt.bezeichnung as gehaltstyp_bezeichnung,
+	vb.vertragsbestandteiltyp_kurzbz,
+	bf.funktion_kurzbz,
+	bf.oe_kurzbz,
+	fkt.beschreibung as fkt_beschreibung,
+	fb.bezeichnung as fb_bezeichnung,
+	org.bezeichnung as org_bezeichnung,
+	freitext.freitexttyp_kurzbz,
+	freitext.titel as freitext_titel
+from
+	gbt
+LEFT JOIN 
+	hr.tbl_gehaltstyp gt using(gehaltstyp_kurzbz)
+LEFT JOIN 
+	hr.tbl_vertragsbestandteil vb using(vertragsbestandteil_id)
+LEFT JOIN 
+	hr.tbl_vertragsbestandteil_funktion vbf using(vertragsbestandteil_id)
+LEFT JOIN 
+	public.tbl_benutzerfunktion bf using(benutzerfunktion_id)
+LEFT JOIN 
+	public.tbl_funktion fkt using(funktion_kurzbz)
+LEFT JOIN 
+	public.tbl_fachbereich fb using(fachbereich_kurzbz)
+LEFT JOIN 
+	public.tbl_organisationseinheit org on (bf.oe_kurzbz=org.oe_kurzbz)
+LEFT JOIN 
+	hr.tbl_vertragsbestandteil_freitext freitext on(vb.vertragsbestandteil_id=freitext.vertragsbestandteil_id)
         ";
 
         return $this->execQuery($qry,
-			array($dienstverhaeltnis_id, $datestring, $datestring),
+			array($dienstverhaeltnis_id, $datestring, $dienstverhaeltnis_id, $datestring),
 			$this->getEncryptedColumns());
     }
 
@@ -86,9 +129,38 @@ class Gehaltsbestandteil_model extends DB_Model implements IEncryption
 			array($dienstverhaeltnis_id),
 			$this->getEncryptedColumns());
     }
-
 	
-	public function getGehaltsbestandteile($dienstverhaeltnis_id, $stichtag=null, $includefuture=false)
+	public function getGehaltsbestandteile($dienstverhaeltnis_id, $stichtag=null, 
+		$includefuture=false, $withvalorisationhistory=true)
+	{
+		if( !is_null($stichtag) && (time() > strtotime($stichtag)) 
+			&& $withvalorisationhistory !== false ) 
+		{
+			$query = $this->getGehaltsbestandteileMitValorisierungsHistorie(
+				$dienstverhaeltnis_id, $stichtag, $includefuture
+			);
+		}
+		else
+		{
+			$query = $this->getGehaltsbestandteileOhneValorisierungsHistorie(
+				$dienstverhaeltnis_id, $stichtag, $includefuture
+			);
+		}
+		
+		$gehaltsbestandteile = array(); 
+		if( null !== ($rows = getData($query)) )
+		{
+			foreach( $rows as $row ) {
+				$tmpgb = new Gehaltsbestandteil();
+				$tmpgb->hydrateByStdClass($row, true);
+				$gehaltsbestandteile[] = $tmpgb;
+			}
+		}
+
+		return $gehaltsbestandteile;
+	}
+
+	protected function getGehaltsbestandteileOhneValorisierungsHistorie($dienstverhaeltnis_id, $stichtag=null, $includefuture=false)
 	{
 		$stichtagclause = '';
 		if( !is_null($stichtag) )
@@ -111,22 +183,64 @@ class Gehaltsbestandteil_model extends DB_Model implements IEncryption
 				{$stichtagclause}
 EOSQL;
 
-		$query = $this->loadWhere(
+		$result = $this->loadWhere(
 			$where,
 			$this->getEncryptedColumns()
 		);
-
-		$gehaltsbestandteile = array(); 
-		if( null !== ($rows = getData($query)) )
-		{
-			foreach( $rows as $row ) {
-				$tmpgb = new Gehaltsbestandteil();
-				$tmpgb->hydrateByStdClass($row, true);
-				$gehaltsbestandteile[] = $tmpgb;
-			}
-		}
-
-		return $gehaltsbestandteile;
+		return $result;
+	}
+	
+	protected function getGehaltsbestandteileMitValorisierungsHistorie($dienstverhaeltnis_id, $stichtag, $includefuture=false)
+	{
+		$date = strftime('%Y-%m-%d', strtotime($stichtag));
+		$includefuture_clause = ($includefuture) 
+			? ' OR COALESCE(von, \'1970-01-01\'::date) > ' . $this->escape($date) 
+			: '';
+		$sql = <<<EOSQL
+SELECT    
+	g.gehaltsbestandteil_id, 
+	g.dienstverhaeltnis_id, 
+	g.vertragsbestandteil_id, 
+	g.gehaltstyp_kurzbz, 
+	g.von, 
+	g.bis, 
+	g.anmerkung, 
+	g.grundbetrag AS grundbetrag, 
+    COALESCE(vh.betrag_valorisiert, g.grundbetrag) AS betrag_valorisiert, 
+	g.valorisierungssperre, 
+	g.insertamum, 
+	g.insertvon, 
+	g.updateamum, 
+	g.updatevon, 
+	g.valorisierung, 
+	g.auszahlungen 
+FROM 
+	hr.tbl_gehaltsbestandteil g 
+LEFT JOIN 
+	hr.tbl_valorisierung_historie vh ON vh.gehaltsbestandteil_id = g.gehaltsbestandteil_id AND vh.valorisierungsdatum = (
+          SELECT 
+          	vi.valorisierungsdatum 
+          FROM 
+          	hr.tbl_valorisierung_instanz vi
+          JOIN
+            hr.tbl_dienstverhaeltnis d ON d.dienstverhaeltnis_id = {$this->escape($dienstverhaeltnis_id)} 
+			AND d.oe_kurzbz = vi.oe_kurzbz
+          WHERE 
+          	{$this->escape($date)} >= valorisierungsdatum 
+          ORDER BY 
+          	valorisierungsdatum DESC 
+          LIMIT 1
+        )
+WHERE 
+	g.dienstverhaeltnis_id = {$this->escape($dienstverhaeltnis_id)} 
+	AND (
+		{$this->escape($date)} BETWEEN COALESCE(von, '1970-01-01'::date) AND COALESCE(bis, '2170-01-01'::date)
+		{$includefuture_clause}
+	)
+EOSQL;
+	
+		$result = $this->execReadOnlyQuery($sql, array(), $this->getEncryptedColumns());
+		return $result;
 	}
 
 	public function getGehaltsbestandteileValorisiert($dienstverhaeltnis_id, $stichtag=null, $includefuture=false)
@@ -159,20 +273,9 @@ EOSQL;
 			ORDER BY gb.von,vh.valorisierungsdatum, gb.gehaltsbestandteil_id;
         ";
 
-		$encryptedColumns = array(
-			'gb.grundbetrag' => array(
-				DB_Model::CRYPT_CAST => 'numeric',
-				DB_Model::CRYPT_PASSWORD_NAME => 'ENCRYPTIONKEYGEHALT'
-			),
-			'vh.betrag_valorisiert' => array(
-				DB_Model::CRYPT_CAST => 'numeric',
-				DB_Model::CRYPT_PASSWORD_NAME => 'ENCRYPTIONKEYGEHALT'
-			)
-		);
-
         $query = $this->execQuery($qry,
 			array($dienstverhaeltnis_id),
-			$encryptedColumns);
+			$this->getEncryptedColumns());
 
 		$gehaltsbestandteile = array();
 		if( null !== ($rows = getData($query)) )
@@ -219,7 +322,6 @@ EOSQL;
 		return $gehaltsbestandteile;
 	}
 
-
 	public function getGehaltsbestandteil($id)
 	{	
 		$this->addSelect('*');

application/models/vertragsbestandteil/Vertragsbestandteil_model.php

@@ -19,12 +19,17 @@ class Vertragsbestandteil_model extends DB_Model
 
 	protected function getVertragsbestandteilSQL()
 	{
+		$sapInstalled = $this->_checkIfSAPSyncTableExists();
+
+		$oe_kurzbz_sap = $sapInstalled ? 'sap.oe_kurzbz_sap' : 'NULL AS oe_kurzbz_sap';
+		$sap_join = $sapInstalled ? 'LEFT JOIN sync.tbl_sap_organisationsstruktur sap USING(oe_kurzbz)' : '';
+
 		$sql = <<<EOSQL
 		SELECT
 				v.*,
-				bf.funktion_kurzbz, bf.uid AS mitarbeiter_uid, 
-				funktion.beschreibung AS funktion_bezeichnung, 			    
-				oe.oe_kurzbz, oe.bezeichnung AS oe_bezeichnung, sap.oe_kurzbz_sap,
+				bf.funktion_kurzbz, bf.uid AS mitarbeiter_uid,
+				funktion.beschreibung AS funktion_bezeichnung,
+				oe.oe_kurzbz, oe.bezeichnung AS oe_bezeichnung, {$oe_kurzbz_sap},
 				oet.organisationseinheittyp_kurzbz AS oe_typ_kurzbz, oet.bezeichnung AS oe_typ_bezeichnung,
 				ft.freitexttyp_kurzbz, ft.titel, ft.anmerkung,
 				f.benutzerfunktion_id,
@@ -39,7 +44,7 @@ class Vertragsbestandteil_model extends DB_Model
 				hr.tbl_vertragsbestandteil_freitext ft USING(vertragsbestandteil_id)
 			LEFT JOIN
 				hr.tbl_vertragsbestandteil_funktion f USING(vertragsbestandteil_id)
-			LEFT JOIN 
+			LEFT JOIN
 				public.tbl_benutzerfunktion bf USING(benutzerfunktion_id)
 			LEFT JOIN
 				public.tbl_funktion funktion USING(funktion_kurzbz)
@@ -47,8 +52,7 @@ class Vertragsbestandteil_model extends DB_Model
 				public.tbl_organisationseinheit oe USING(oe_kurzbz)
 			LEFT JOIN
 				public.tbl_organisationseinheittyp oet USING(organisationseinheittyp_kurzbz)
-			LEFT JOIN
-				sync.tbl_sap_organisationsstruktur sap USING(oe_kurzbz)
+			{$sap_join}
 			LEFT JOIN
 				hr.tbl_vertragsbestandteil_karenz k USING(vertragsbestandteil_id)
 			LEFT JOIN
@@ -178,4 +182,30 @@ EOSQL;
 		
 		return $vbcount[0]->overlappingvbs;
 	}
+
+	/**
+	 * Checks if sap sync table exists.
+	 * @return bool
+	 */
+	private function _checkIfSAPSyncTableExists()
+	{
+		$params = array(
+			DB_NAME,
+			'sync',
+			'tbl_sap_organisationsstruktur'
+		);
+
+		$sql = "SELECT
+				1 AS exists
+			FROM
+				information_schema.tables
+			WHERE
+				table_catalog = ? AND
+				table_schema = ? AND
+				table_name = ?";
+
+		$res = $this->execReadOnlyQuery($sql, $params);
+
+		return hasData($res);
+	}
 }