Compare commits
5 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 3e0cf10d6c | |||
| 524f293e37 | |||
| 5ba4029912 | |||
| 305ee59477 | |||
| 7fde8cb3d9 |
@@ -2,13 +2,27 @@
|
||||
* FinanceBuddy styles.
|
||||
*/
|
||||
|
||||
/* Per-column net total in the column header (its own line, subtle pill). */
|
||||
/* The column header (.board-column-expanded-header) is a flex row, so the total is a flex item. To
|
||||
drop it onto its own line under the title, let that header wrap (scoped with :has to headers that
|
||||
actually contain the total -- i.e. enabled boards only) and give the total full width. No
|
||||
background (dropped per request); "Total:" stays black, only the amount span is colored by sign. */
|
||||
.board-column-expanded-header:has(.fb-column-total) {
|
||||
flex-wrap: wrap;
|
||||
}
|
||||
|
||||
.fb-column-total {
|
||||
flex-basis: 100%;
|
||||
margin-top: 2px;
|
||||
text-align: right;
|
||||
}
|
||||
|
||||
/* The visible pill hugs the text (not the full width) and is right-justified on its line. */
|
||||
.fb-total-pill {
|
||||
display: inline-block;
|
||||
margin-top: 3px;
|
||||
padding: 0 5px;
|
||||
font-weight: bold;
|
||||
font-size: 0.95em;
|
||||
color: #000;
|
||||
background: rgba(0, 0, 0, 0.05);
|
||||
border-radius: 3px;
|
||||
}
|
||||
|
||||
@@ -73,6 +73,19 @@ class FinancePersistence extends Base
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* recoreco:task:spawned -- RecoReco (calendar recurrence) just spawned a copy. Its ongoing card
|
||||
* is the TEMPLATE that stays (unlike native recurrence, whose ongoing card is the new child), so
|
||||
* advance the SOURCE template's installment. The clone already copied the current installment
|
||||
* verbatim via onDuplication during RecoReco's duplicate() call.
|
||||
*/
|
||||
public function onRecoRecoSpawn(array &$payload)
|
||||
{
|
||||
if (! empty($payload['source_task_id'])) {
|
||||
$this->advanceInstallment((int) $payload['source_task_id']);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* model:task:modification:prepare -- fires before UPDATE, $values by reference. The task id is
|
||||
* removed from $values by core, but it is the route parameter of the modification request.
|
||||
@@ -176,6 +189,31 @@ class FinancePersistence extends Base
|
||||
));
|
||||
}
|
||||
|
||||
/**
|
||||
* Advance a task's current installment by one, in place. A finite plan clamps at the total
|
||||
* (p10/10 stays p10/10); an open-ended one (no total) increments unbounded (p2 -> p3). Cards
|
||||
* with no numeric current installment are left alone.
|
||||
*/
|
||||
private function advanceInstallment($task_id)
|
||||
{
|
||||
$current = $this->taskMetadataModel->get($task_id, FinanceHelper::INSTALLMENT_CURRENT_KEY, '');
|
||||
|
||||
if ($current === '' || ! ctype_digit((string) $current)) {
|
||||
return;
|
||||
}
|
||||
|
||||
$total = $this->taskMetadataModel->get($task_id, FinanceHelper::INSTALLMENT_TOTAL_KEY, '');
|
||||
$next = (int) $current + 1;
|
||||
|
||||
if ($total !== '' && ctype_digit((string) $total) && $next > (int) $total) {
|
||||
$next = (int) $total;
|
||||
}
|
||||
|
||||
$this->taskMetadataModel->save($task_id, array(
|
||||
FinanceHelper::INSTALLMENT_CURRENT_KEY => (string) $next,
|
||||
));
|
||||
}
|
||||
|
||||
private function intOrBlank($raw)
|
||||
{
|
||||
$raw = trim((string) $raw);
|
||||
|
||||
@@ -34,6 +34,11 @@ class Plugin extends Base
|
||||
// and a manual Duplicate copies it verbatim.
|
||||
$this->hook->on('model:task:duplication:aftersave', array($persistence, 'onDuplication'));
|
||||
|
||||
// RecoReco (calendar recurrence) spawns a copy and emits this signal. The clone already got
|
||||
// the finance verbatim (via onDuplication above); here we advance the SOURCE template's
|
||||
// installment, since RecoReco's ongoing card is the template that stays.
|
||||
$this->hook->on('recoreco:task:spawned', array($persistence, 'onRecoRecoSpawn'));
|
||||
|
||||
// The line-3 money badge on each board card, rendered directly under the title. Two hooks:
|
||||
// "private" is the normal board, "public" is the read-only board shared by token.
|
||||
$this->template->hook->attach('template:board:private:task:after-title', 'financeBuddy:board/task_money');
|
||||
@@ -72,7 +77,7 @@ class Plugin extends Base
|
||||
|
||||
public function getPluginVersion()
|
||||
{
|
||||
return '1.0.1';
|
||||
return '1.0.5';
|
||||
}
|
||||
|
||||
public function getPluginHomepage()
|
||||
|
||||
@@ -1,7 +1,8 @@
|
||||
<?php
|
||||
// Per-column net total in the column header (credits - debits). Shown in every column of an
|
||||
// enabled board, including "total R$0,00" -- never hidden on zero. Server-side, re-renders on
|
||||
// every board refresh.
|
||||
// enabled board, including "Total: R$0,00" -- never hidden on zero. Server-side, re-renders on
|
||||
// every board refresh. Rendered on its own full-width line under the title; "Total:" stays the
|
||||
// default (black) and only the amount is colored by sign.
|
||||
if (empty($column['project_id']) || ! $this->FinanceHelper->isEnabled($column['project_id'])) {
|
||||
return;
|
||||
}
|
||||
@@ -9,16 +10,16 @@ if (empty($column['project_id']) || ! $this->FinanceHelper->isEnabled($column['p
|
||||
$net = $this->FinanceHelper->columnNet($column);
|
||||
|
||||
if ($net > 0) {
|
||||
$class = 'fb-column-total fb-total-credit';
|
||||
$amount_class = 'fb-total-credit';
|
||||
$sign = '+';
|
||||
} elseif ($net < 0) {
|
||||
$class = 'fb-column-total fb-total-debit';
|
||||
$amount_class = 'fb-total-debit';
|
||||
$sign = '-';
|
||||
} else {
|
||||
$class = 'fb-column-total fb-total-zero';
|
||||
$amount_class = 'fb-total-zero';
|
||||
$sign = '';
|
||||
}
|
||||
?>
|
||||
<div class="<?= $class ?>">
|
||||
<?= t('Total') ?>: <span class="fb-amount"><?= $sign ?>R$<?= \Kanboard\Plugin\FinanceBuddy\Helper\FinanceHelper::money(abs($net)) ?></span>
|
||||
<div class="fb-column-total">
|
||||
<span class="fb-total-pill"><?= t('Total') ?>: <span class="fb-amount <?= $amount_class ?>"><?= $sign ?>R$<?= \Kanboard\Plugin\FinanceBuddy\Helper\FinanceHelper::money(abs($net)) ?></span></span>
|
||||
</div>
|
||||
|
||||
@@ -15,13 +15,18 @@ if ($fb['amount'] === '' || (float) $fb['amount'] <= 0) {
|
||||
$credit = $fb['direction'] === 'credit';
|
||||
$sign = $credit ? '+' : '-';
|
||||
$total = $fb['installment_total'];
|
||||
$current = $fb['installment_current'] !== '' ? $fb['installment_current'] : '1';
|
||||
$current = $fb['installment_current'];
|
||||
|
||||
// Installments: "p2/10" when a total is set; "p2" (open-ended) when only the current is set;
|
||||
// nothing when neither is set. Default the current to 1 if only a total was entered.
|
||||
$show_installments = $current !== '' || $total !== '';
|
||||
$current_display = $current !== '' ? $current : '1';
|
||||
?>
|
||||
<div class="fb-line">
|
||||
<span class="fb-badge <?= $credit ? 'fb-badge-credit' : 'fb-badge-debit' ?>">
|
||||
<?= $sign ?>R$<?= \Kanboard\Plugin\FinanceBuddy\Helper\FinanceHelper::money($fb['amount']) ?>
|
||||
</span>
|
||||
<?php if ($total !== ''): ?>
|
||||
<span class="fb-installments">p<?= $this->text->e($current) ?>/<?= $this->text->e($total) ?></span>
|
||||
<?php if ($show_installments): ?>
|
||||
<span class="fb-installments">p<?= $this->text->e($current_display) ?><?= $total !== '' ? '/'.$this->text->e($total) : '' ?></span>
|
||||
<?php endif ?>
|
||||
</div>
|
||||
|
||||
@@ -24,6 +24,6 @@
|
||||
<span><?= t('of') ?></span>
|
||||
<input type="number" min="1" name="financebuddy_installment_total" value="<?= $this->text->e($fb['installment_total']) ?>" class="fb-form-num">
|
||||
</div>
|
||||
<p class="form-help"><?= t('Leave the total blank for an open-ended recurring bill.') ?></p>
|
||||
<p class="form-help"><?= t('Set a total for a finite plan (shows p2/10); leave it blank for an open-ended count (shows p2).') ?></p>
|
||||
</div>
|
||||
<?php endif ?>
|
||||
|
||||
Reference in New Issue
Block a user