intraface /
intraface.dk
This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
|||||||||||
| 2 | require_once 'Intraface/modules/accounting/VatPeriod.php'; |
|||||||||||
| 3 | require_once 'Intraface/modules/accounting/Account.php'; |
|||||||||||
| 4 | require_once 'Intraface/modules/accounting/Voucher.php'; |
|||||||||||
| 5 | require_once 'Intraface/Kernel.php'; |
|||||||||||
| 6 | ||||||||||||
| 7 | class FakeVatPeriodYear |
|||||||||||
| 8 | { |
|||||||||||
| 9 | public $kernel; |
|||||||||||
| 10 | function __construct() |
|||||||||||
| 11 | { |
|||||||||||
| 12 | $this->kernel = new Stub_Kernel; |
|||||||||||
| 13 | $this->kernel->setting->set('intranet', 'vatpercent', 25); |
|||||||||||
| 14 | } |
|||||||||||
| 15 | function get() |
|||||||||||
| 16 | { |
|||||||||||
| 17 | return 1; |
|||||||||||
| 18 | } |
|||||||||||
| 19 | ||||||||||||
| 20 | function getSetting($key = '') |
|||||||||||
| 21 | { |
|||||||||||
| 22 | if ($key == 'vat_period') { |
|||||||||||
| 23 | return 1; |
|||||||||||
| 24 | } |
|||||||||||
| 25 | return serialize(array()); |
|||||||||||
| 26 | } |
|||||||||||
| 27 | ||||||||||||
| 28 | function isYearOpen() |
|||||||||||
| 29 | { |
|||||||||||
| 30 | return true; |
|||||||||||
| 31 | } |
|||||||||||
| 32 | ||||||||||||
| 33 | function isDateInYear() |
|||||||||||
| 34 | { |
|||||||||||
| 35 | return true; |
|||||||||||
| 36 | } |
|||||||||||
| 37 | } |
|||||||||||
| 38 | ||||||||||||
| 39 | class FakeVatPeriodAccount extends Account |
|||||||||||
| 40 | { |
|||||||||||
| 41 | protected $id; |
|||||||||||
| 42 | ||||||||||||
| 43 | function __construct($year, $id = 0) |
|||||||||||
| 44 | { |
|||||||||||
| 45 | $this->id = (int)$id; |
|||||||||||
| 46 | $this->value['number'] = 1000; |
|||||||||||
| 47 | $this->value['saldo'] = 100; |
|||||||||||
| 48 | } |
|||||||||||
| 49 | ||||||||||||
| 50 | function getSaldo($type, $date_from, $date_to) |
|||||||||||
| 51 | { |
|||||||||||
| 52 | } |
|||||||||||
| 53 | ||||||||||||
| 54 | function getId() |
|||||||||||
| 55 | { |
|||||||||||
| 56 | return $this->id; |
|||||||||||
| 57 | } |
|||||||||||
| 58 | ||||||||||||
| 59 | function validForState() |
|||||||||||
| 60 | { |
|||||||||||
| 61 | return true; |
|||||||||||
| 62 | } |
|||||||||||
| 63 | } |
|||||||||||
| 64 | ||||||||||||
| 65 | class FakeVatPeriodVoucher extends Voucher |
|||||||||||
| 66 | { |
|||||||||||
| 67 | function getAccount($id) |
|||||||||||
| 68 | { |
|||||||||||
| 69 | return new FakeVatPeriodAccount(new FakeVatPeriodYear); |
|||||||||||
| 70 | } |
|||||||||||
| 71 | } |
|||||||||||
| 72 | ||||||||||||
| 73 | class TestableVatPeriod extends VatPeriod |
|||||||||||
| 74 | { |
|||||||||||
| 75 | function getId() |
|||||||||||
| 76 | { |
|||||||||||
| 77 | return 1; |
|||||||||||
| 78 | } |
|||||||||||
| 79 | ||||||||||||
| 80 | function getAccount($id) |
|||||||||||
| 81 | { |
|||||||||||
| 82 | return new FakeVatPeriodAccount(new FakeVatPeriodYear, $id); |
|||||||||||
| 83 | } |
|||||||||||
| 84 | ||||||||||||
| 85 | function getVoucher() |
|||||||||||
| 86 | { |
|||||||||||
| 87 | return new FakeVatPeriodVoucher(new FakeVatPeriodYear); |
|||||||||||
| 88 | } |
|||||||||||
| 89 | ||||||||||||
| 90 | function loadAmounts() |
|||||||||||
| 91 | { |
|||||||||||
| 92 | $saldo_total = 0; // integer med total saldo |
|||||||||||
| 93 | $saldo_rubrik_a = 0; |
|||||||||||
|
0 ignored issues
–
show
|
||||||||||||
| 94 | ||||||||||||
| 95 | $date_from = $this->get('date_start'); |
|||||||||||
| 96 | $date_to = $this->get('date_end'); |
|||||||||||
| 97 | ||||||||||||
| 98 | // Salgsmoms - udg�ende |
|||||||||||
| 99 | $account_vat_in = $this->getAccount(1); |
|||||||||||
| 100 | $account_vat_in->getSaldo('stated', $date_from, $date_to); |
|||||||||||
| 101 | $this->value['account_vat_out'] = $account_vat_in; |
|||||||||||
| 102 | ||||||||||||
| 103 | // ganges med -1 for at f� rigtigt fortegn til udregning |
|||||||||||
| 104 | $this->value['saldo_vat_out'] = $account_vat_in->get('saldo'); |
|||||||||||
| 105 | $saldo_total += -1 * $this->value['saldo_vat_out']; // total |
|||||||||||
| 106 | ||||||||||||
| 107 | ||||||||||||
| 108 | // Moms af varek�b i udlandet |
|||||||||||
| 109 | // Dette bel�b er et udregnet bel�b, som udregnes under bogf�ringen |
|||||||||||
| 110 | $account_vat_abroad = $this->getAccount(2); |
|||||||||||
| 111 | $account_vat_abroad->getSaldo('stated', $date_from, $date_to); |
|||||||||||
| 112 | $this->value['account_vat_abroad'] = $account_vat_abroad; |
|||||||||||
| 113 | ||||||||||||
| 114 | // ganges med -1 for at f� rigtigt fortegn til udregning |
|||||||||||
| 115 | $this->value['saldo_vat_abroad'] = $account_vat_abroad->get('saldo'); |
|||||||||||
| 116 | $saldo_total += -1 * $this->value['saldo_vat_abroad']; |
|||||||||||
| 117 | ||||||||||||
| 118 | // K�bsmoms |
|||||||||||
| 119 | // K�bsmomsen inkluderer ogs� den udregnede moms af moms af varek�b i udlandet. |
|||||||||||
| 120 | // Dette bel�b er lagt p� kontoen under bogf�ringen. |
|||||||||||
| 121 | $account_vat_out = $this->getAccount(3); |
|||||||||||
| 122 | $account_vat_out->getSaldo('stated', $date_from, $date_to); |
|||||||||||
| 123 | $this->value['account_vat_in'] = $account_vat_out; |
|||||||||||
| 124 | ||||||||||||
| 125 | $this->value['saldo_vat_in'] = $account_vat_out->get('saldo'); |
|||||||||||
| 126 | $saldo_total -= $this->value['saldo_vat_in']; |
|||||||||||
| 127 | ||||||||||||
| 128 | ||||||||||||
| 129 | // Rubrik A |
|||||||||||
| 130 | // EU-erhvervelser - her samles forskellige konti og bel�bet udregnes. |
|||||||||||
| 131 | // Primosaldoen skal ikke medregnes |
|||||||||||
| 132 | $buy_eu_accounts = unserialize($this->year->getSetting('buy_eu_accounts')); |
|||||||||||
| 133 | $this->value['saldo_rubrik_a'] = 0; |
|||||||||||
| 134 | $saldo_rubrik_a = 0; |
|||||||||||
| 135 | ||||||||||||
| 136 | View Code Duplication | if (is_array($buy_eu_accounts) and count($buy_eu_accounts) > 0) { |
||||||||||
|
0 ignored issues
–
show
Comprehensibility
Best Practice
introduced
by
Using logical operators such as
and instead of && is generally not recommended.
PHP has two types of connecting operators (logical operators, and boolean operators):
The difference between these is the order in which they are executed. In most cases,
you would want to use a boolean operator like Let’s take a look at a few examples: // Logical operators have lower precedence:
$f = false or true;
// is executed like this:
($f = false) or true;
// Boolean operators have higher precedence:
$f = false || true;
// is executed like this:
$f = (false || true);
Logical Operators are used for Control-FlowOne case where you explicitly want to use logical operators is for control-flow such as this: $x === 5
or die('$x must be 5.');
// Instead of
if ($x !== 5) {
die('$x must be 5.');
}
Since // The following is currently a parse error.
$x === 5
or throw new RuntimeException('$x must be 5.');
These limitations lead to logical operators rarely being of use in current PHP code. Loading history...
|
||||||||||||
| 137 | foreach ($buy_eu_accounts as $key => $id) { |
|||||||||||
| 138 | $account_eu_buy = new FakeVatPeriodAccount($this->year, $id); |
|||||||||||
| 139 | $primo = $account_eu_buy->getPrimoSaldo(); |
|||||||||||
| 140 | $account_eu_buy->getSaldo('stated', $date_from, $date_to); |
|||||||||||
| 141 | $saldo_rubrik_a += $account_eu_buy->get('saldo'); |
|||||||||||
| 142 | $saldo_rubrik_a -= $primo['saldo']; |
|||||||||||
| 143 | } |
|||||||||||
| 144 | } |
|||||||||||
| 145 | $this->value['saldo_rubrik_a'] = $saldo_rubrik_a; |
|||||||||||
| 146 | ||||||||||||
| 147 | // Rubrik B |
|||||||||||
| 148 | // V�rdien af varesalg uden moms til andre EU-lande (EU-leverancer). Udfyldes |
|||||||||||
| 149 | // denne rubrik, skal der indsendes en liste med varesalgene uden moms. |
|||||||||||
| 150 | ||||||||||||
| 151 | // Vi underst�tter ikke rubrikken |
|||||||||||
| 152 | ||||||||||||
| 153 | // Rubrik C |
|||||||||||
| 154 | // V�rdien af varesalg uden moms til andre EU-lande (EU-leverancer). Udfyldes |
|||||||||||
| 155 | // denne rubrik, skal der indsendes en liste med varesalgene uden moms. |
|||||||||||
| 156 | ||||||||||||
| 157 | // Vi underst�tter ikke rubrikken |
|||||||||||
| 158 | ||||||||||||
| 159 | $this->value['saldo_total'] = $saldo_total; |
|||||||||||
| 160 | ||||||||||||
| 161 | return true; |
|||||||||||
| 162 | } |
|||||||||||
| 163 | } |
|||||||||||
| 164 | ||||||||||||
| 165 | class VatPeriodTest extends PHPUnit_Framework_TestCase |
|||||||||||
| 166 | { |
|||||||||||
| 167 | protected $db; |
|||||||||||
| 168 | ||||||||||||
| 169 | function setUp() |
|||||||||||
| 170 | { |
|||||||||||
| 171 | $this->db = MDB2::singleton(DB_DSN); |
|||||||||||
| 172 | } |
|||||||||||
| 173 | ||||||||||||
| 174 | function tearDown() |
|||||||||||
| 175 | { |
|||||||||||
| 176 | $this->db->exec('TRUNCATE accounting_account'); |
|||||||||||
| 177 | } |
|||||||||||
| 178 | ||||||||||||
| 179 | function createPeriod() |
|||||||||||
| 180 | { |
|||||||||||
| 181 | return new VatPeriod(new FakeVatPeriodYear); |
|||||||||||
| 182 | } |
|||||||||||
| 183 | ||||||||||||
| 184 | function createPeriodWithFakeLoadAmounts() |
|||||||||||
| 185 | { |
|||||||||||
| 186 | return new TestableVatPeriod(new FakeVatPeriodYear); |
|||||||||||
| 187 | } |
|||||||||||
| 188 | ||||||||||||
| 189 | function testConstruction() |
|||||||||||
| 190 | { |
|||||||||||
| 191 | $vat = $this->createPeriod(); |
|||||||||||
| 192 | $this->assertTrue(is_object($vat)); |
|||||||||||
| 193 | } |
|||||||||||
| 194 | ||||||||||||
| 195 | function testCreatePeriods() |
|||||||||||
| 196 | { |
|||||||||||
| 197 | $vat = $this->createPeriod(); |
|||||||||||
| 198 | $this->assertTrue($vat->createPeriods()); |
|||||||||||
| 199 | $periods = $vat->getList(); |
|||||||||||
| 200 | $this->assertTrue(count($periods) == 4); |
|||||||||||
| 201 | $this->assertTrue($periods[0]['date_start'] == '0001-01-01' && $periods[0]['date_end'] == '0001-03-31'); |
|||||||||||
| 202 | $this->assertTrue($periods[1]['date_start'] == '0001-04-01' && $periods[1]['date_end'] == '0001-06-30'); |
|||||||||||
| 203 | $this->assertTrue($periods[2]['date_start'] == '0001-07-01' && $periods[2]['date_end'] == '0001-09-30'); |
|||||||||||
| 204 | $this->assertTrue($periods[3]['date_start'] == '0001-10-01' && $periods[3]['date_end'] == '0001-12-31'); |
|||||||||||
| 205 | } |
|||||||||||
| 206 | ||||||||||||
| 207 | function testStateReturnsTrue() |
|||||||||||
| 208 | { |
|||||||||||
| 209 | // @todo make sure that the getSetting returns some usable accounts. |
|||||||||||
| 210 | $vat = $this->createPeriodWithFakeLoadAmounts(); |
|||||||||||
| 211 | $this->assertTrue($vat->state(date('d-m-Y'), 100)); |
|||||||||||
| 212 | } |
|||||||||||
| 213 | ||||||||||||
| 214 | function testIsBalancedReturnsTrueWhenNoAmountsHasBeenSaved() |
|||||||||||
| 215 | { |
|||||||||||
| 216 | $vat = $this->createPeriod(); |
|||||||||||
| 217 | $this->assertTrue($vat->isBalanced()); |
|||||||||||
| 218 | } |
|||||||||||
| 219 | ||||||||||||
| 220 | function testDeleteReturnsTrue() |
|||||||||||
| 221 | { |
|||||||||||
| 222 | $vat = $this->createPeriod(); |
|||||||||||
| 223 | $this->assertTrue($vat->delete()); |
|||||||||||
| 224 | } |
|||||||||||
| 225 | } |
|||||||||||
| 226 |
This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently.
Both the
$myVarassignment in line 1 and the$higherassignment in line 2 are dead. The first because$myVaris never used and the second because$higheris always overwritten for every possible time line.