forked from fixin.me/fixin.me
Compare commits
1 Commits
feature/me
...
feature/mu
| Author | SHA1 | Date | |
|---|---|---|---|
| 24539f236c |
84
CLAUDE.md
84
CLAUDE.md
@@ -1,84 +0,0 @@
|
|||||||
# CLAUDE.md
|
|
||||||
|
|
||||||
This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
|
|
||||||
|
|
||||||
## Project Overview
|
|
||||||
|
|
||||||
Fixin.me is a "quantified self" Rails 7.2.3 application for personal data tracking. Users define hierarchical **quantities** (metrics to track), **units** (with optional conversion hierarchies), and **readouts** (individual measurements). There is also a non-persistent **measurement** model used as a form wrapper.
|
|
||||||
|
|
||||||
## Setup
|
|
||||||
|
|
||||||
Configuration files are distributed as `.dist` templates — copy and customize before use:
|
|
||||||
|
|
||||||
```bash
|
|
||||||
cp config/application.rb.dist config/application.rb
|
|
||||||
cp config/database.yml.dist config/database.yml
|
|
||||||
cp config/puma.rb.dist config/puma.rb
|
|
||||||
```
|
|
||||||
|
|
||||||
```bash
|
|
||||||
bundle config --local frozen true
|
|
||||||
bundle config --local path .gem
|
|
||||||
bundle config --local with mysql development test # or: pg, sqlite
|
|
||||||
bundle install
|
|
||||||
RAILS_ENV=development bundle exec rails db:create db:migrate db:seed
|
|
||||||
```
|
|
||||||
|
|
||||||
## Common Commands
|
|
||||||
|
|
||||||
```bash
|
|
||||||
bundle exec rails s # start server
|
|
||||||
bundle exec rails test # all unit/model/controller tests
|
|
||||||
bundle exec rails test:system # all system tests (Capybara + Selenium)
|
|
||||||
bundle exec rails test test/system/units_test.rb # single test file
|
|
||||||
bundle exec rails test --seed 64690 --name test_add_unit # single test by name
|
|
||||||
bundle exec rails db:seed:export # export default settings as seed file
|
|
||||||
```
|
|
||||||
|
|
||||||
## Architecture
|
|
||||||
|
|
||||||
### Data Model
|
|
||||||
|
|
||||||
- **Quantity** — hierarchical tree (self-referential `parent_id`). Cached `depth` and `pathname` fields are recomputed via recursive CTEs on write. Direct assignment to cached fields is blocked.
|
|
||||||
- **Unit** — optional hierarchy via `base_id` and `multiplier` for unit conversion. Multiplier precision/scale is validated by a custom validator.
|
|
||||||
- **Readout** — single measurement: `value` (IEEE 754 float), `quantity`, `unit`, `category`.
|
|
||||||
- **Measurement** — `ActiveModel::Model` form wrapper (not database-backed); bridges the readout creation form.
|
|
||||||
- **User** — Devise-managed with a status enum: `admin`, `active`, `restricted`, `locked`, `disabled`. Admins can disguise as other users.
|
|
||||||
|
|
||||||
### Hierarchical Queries
|
|
||||||
|
|
||||||
Both `Quantity` and `Unit` use recursive CTEs for tree traversal (ordered traversal, ancestors, progenies, common ancestors). `lib/core_ext/arel/` patches Arel to support CTE with `UPDATE`/`DELETE` statements, working around Rails issue #54658.
|
|
||||||
|
|
||||||
### Custom Extensions (`lib/core_ext/`)
|
|
||||||
|
|
||||||
- **arel/** — CTE support for UPDATE/DELETE
|
|
||||||
- **active_model/** — precision/scale validator used by `Unit#multiplier`
|
|
||||||
- **active_record/** — `attr_cached` mechanism (see `ApplicationRecord`)
|
|
||||||
- **action_view/** — record identifier suffixes
|
|
||||||
- Miscellaneous: `Array#delete_bang`, `BigDecimal` scientific notation
|
|
||||||
|
|
||||||
### Response Handling
|
|
||||||
|
|
||||||
Controllers respond to both HTML and Turbo Stream formats. Errors during Turbo Stream requests trigger a redirect with flash rather than rendering inline, handled in `ApplicationController`.
|
|
||||||
|
|
||||||
### Numeric Precision
|
|
||||||
|
|
||||||
Readout values are stored as IEEE 754 double-precision floats (not fixed-point decimals). Rationale in `DESIGN.md`: biological values span many orders of magnitude; 15-digit float precision is sufficient and avoids conversion overhead.
|
|
||||||
|
|
||||||
### Routes
|
|
||||||
|
|
||||||
```
|
|
||||||
measurements GET/POST /measurements
|
|
||||||
readouts GET/POST /readouts, DELETE /readouts/:id/discard
|
|
||||||
quantities CRUD + POST /quantities/:id/reparent
|
|
||||||
units CRUD + POST /units/:id/rebase
|
|
||||||
users CRUD + POST /users/:id/disguise, POST /users/revert
|
|
||||||
default/ namespace for default units import/export and admin panel
|
|
||||||
root → /units (authenticated), /sign_in (unauthenticated)
|
|
||||||
```
|
|
||||||
|
|
||||||
## Database Requirements
|
|
||||||
|
|
||||||
The database must support:
|
|
||||||
- Recursive CTEs with `UPDATE`/`DELETE` (MySQL ≥ 8.0, PostgreSQL, or SQLite3)
|
|
||||||
- Decimal precision of 30+ digits
|
|
||||||
@@ -1 +0,0 @@
|
|||||||
<svg xmlns="http://www.w3.org/2000/svg" id="icon" viewBox="0 0 24 24"><path d="M4,4H8V20H4V4M10,4H14V20H10V4M16,4H21V20H16V4Z"/></svg>
|
|
||||||
|
Before Width: | Height: | Size: 135 B |
@@ -1 +0,0 @@
|
|||||||
<svg xmlns="http://www.w3.org/2000/svg" id="icon" viewBox="0 0 24 24"><path d="M3,5H21V7H3V5M3,11H21V13H3V11M3,17H21V19H3V17Z"/></svg>
|
|
||||||
|
Before Width: | Height: | Size: 135 B |
@@ -36,7 +36,6 @@
|
|||||||
--color-blue: #009ade;
|
--color-blue: #009ade;
|
||||||
--color-dark-red: #b21237;
|
--color-dark-red: #b21237;
|
||||||
--color-red: #ff1f5b;
|
--color-red: #ff1f5b;
|
||||||
--color-purple: #8b2be2;
|
|
||||||
|
|
||||||
--depth: 0;
|
--depth: 0;
|
||||||
|
|
||||||
@@ -232,10 +231,6 @@ textarea:invalid {
|
|||||||
text-decoration: underline 1px var(--color-border-gray);
|
text-decoration: underline 1px var(--color-border-gray);
|
||||||
text-underline-offset: 0.25em;
|
text-underline-offset: 0.25em;
|
||||||
}
|
}
|
||||||
button.link {
|
|
||||||
border: none;
|
|
||||||
padding: 0;
|
|
||||||
}
|
|
||||||
[name=cancel],
|
[name=cancel],
|
||||||
.auxiliary {
|
.auxiliary {
|
||||||
border-color: var(--color-border-gray);
|
border-color: var(--color-border-gray);
|
||||||
@@ -257,13 +252,6 @@ button.link {
|
|||||||
background-color: var(--color-red);
|
background-color: var(--color-red);
|
||||||
border-color: var(--color-red);
|
border-color: var(--color-red);
|
||||||
}
|
}
|
||||||
tr:has(select[data-changed]) button[name="button"],
|
|
||||||
.set-default-unit:not([disabled]) {
|
|
||||||
background-color: var(--color-purple);
|
|
||||||
border-color: var(--color-purple);
|
|
||||||
color: white;
|
|
||||||
fill: white;
|
|
||||||
}
|
|
||||||
.link:focus-visible {
|
.link:focus-visible {
|
||||||
text-decoration-color: var(--color-gray);
|
text-decoration-color: var(--color-gray);
|
||||||
}
|
}
|
||||||
@@ -373,7 +361,7 @@ header {
|
|||||||
pointer-events: auto;
|
pointer-events: auto;
|
||||||
}
|
}
|
||||||
.flash:before {
|
.flash:before {
|
||||||
filter: invert(100%);
|
filter: invert();
|
||||||
height: 1.4em;
|
height: 1.4em;
|
||||||
margin: 0 0.5em;
|
margin: 0 0.5em;
|
||||||
width: 1.4em;
|
width: 1.4em;
|
||||||
@@ -525,7 +513,7 @@ header {
|
|||||||
cursor: grab;
|
cursor: grab;
|
||||||
}
|
}
|
||||||
.items-table .form td {
|
.items-table .form td {
|
||||||
vertical-align: middle;
|
vertical-align: top;
|
||||||
}
|
}
|
||||||
.items-table td:not(:first-child),
|
.items-table td:not(:first-child),
|
||||||
.grayed {
|
.grayed {
|
||||||
@@ -652,42 +640,3 @@ li::marker {
|
|||||||
min-width: 66%;
|
min-width: 66%;
|
||||||
width: max-content;
|
width: max-content;
|
||||||
}
|
}
|
||||||
.measurements-section {
|
|
||||||
overflow-x: auto;
|
|
||||||
}
|
|
||||||
body[data-measurements-view=wide] .measurements-compact,
|
|
||||||
body[data-measurements-view=compact] .measurements-wide {
|
|
||||||
display: none;
|
|
||||||
}
|
|
||||||
body[data-measurements-view=compact] .view-toggle[data-view=compact],
|
|
||||||
body[data-measurements-view=wide] .view-toggle[data-view=wide] {
|
|
||||||
background-color: var(--color-blue);
|
|
||||||
border-color: var(--color-blue);
|
|
||||||
color: white;
|
|
||||||
fill: white;
|
|
||||||
}
|
|
||||||
#measurements tr.grouped td {
|
|
||||||
border-top: none;
|
|
||||||
}
|
|
||||||
#measurements tr.grouped .taken-at,
|
|
||||||
#measurements tr.grouped .created-at {
|
|
||||||
visibility: hidden;
|
|
||||||
}
|
|
||||||
.measurements-wide td {
|
|
||||||
vertical-align: middle;
|
|
||||||
white-space: nowrap;
|
|
||||||
}
|
|
||||||
.wide-cell {
|
|
||||||
align-items: center;
|
|
||||||
display: inline-flex;
|
|
||||||
gap: 0.25em;
|
|
||||||
}
|
|
||||||
.wide-cell .button {
|
|
||||||
border: none;
|
|
||||||
font-size: inherit;
|
|
||||||
height: auto;
|
|
||||||
padding: 0;
|
|
||||||
}
|
|
||||||
.wide-cell button.link::after {
|
|
||||||
content: none;
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -1,12 +1,7 @@
|
|||||||
class MeasurementsController < ApplicationController
|
class MeasurementsController < ApplicationController
|
||||||
before_action :find_readout, only: [:destroy, :edit, :update]
|
|
||||||
|
|
||||||
before_action except: :index do
|
|
||||||
raise AccessForbidden unless current_user.at_least(:active)
|
|
||||||
end
|
|
||||||
|
|
||||||
def index
|
def index
|
||||||
load_measurements
|
@measurements = []
|
||||||
|
#@measurements = current_user.units.ordered.includes(:base, :subunits)
|
||||||
end
|
end
|
||||||
|
|
||||||
def new
|
def new
|
||||||
@@ -14,49 +9,8 @@ class MeasurementsController < ApplicationController
|
|||||||
end
|
end
|
||||||
|
|
||||||
def create
|
def create
|
||||||
taken_at = params.permit(:taken_at)[:taken_at]
|
|
||||||
readout_params = params.permit(readouts: Readout::ATTRIBUTES).fetch(:readouts, [])
|
|
||||||
@readouts = readout_params.map { |rp| current_user.readouts.build(rp.merge(taken_at: taken_at)) }
|
|
||||||
|
|
||||||
if @readouts.present? && @readouts.all?(&:valid?)
|
|
||||||
ActiveRecord::Base.transaction { @readouts.each(&:save!) }
|
|
||||||
load_measurements
|
|
||||||
flash.now[:notice] = t('.success', count: @readouts.size)
|
|
||||||
else
|
|
||||||
errors = @readouts.flat_map { |r| r.errors.full_messages }
|
|
||||||
flash.now[:alert] = errors.present? ? errors.first : t('.no_readouts')
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def edit
|
|
||||||
@user_units = current_user.units.ordered
|
|
||||||
end
|
|
||||||
|
|
||||||
def update
|
|
||||||
if @readout.update(params.require(:readout).permit(:value, :unit_id, :taken_at))
|
|
||||||
load_measurements
|
|
||||||
flash.now[:notice] = t('.success')
|
|
||||||
else
|
|
||||||
@user_units = current_user.units.ordered
|
|
||||||
render :edit
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
def destroy
|
def destroy
|
||||||
@readout.destroy!
|
|
||||||
load_measurements
|
|
||||||
flash.now[:notice] = t('.success')
|
|
||||||
end
|
|
||||||
|
|
||||||
private
|
|
||||||
|
|
||||||
def find_readout
|
|
||||||
@readout = current_user.readouts.find(params[:id])
|
|
||||||
end
|
|
||||||
|
|
||||||
def load_measurements
|
|
||||||
@measurements = current_user.readouts.includes(:quantity, :unit).order(taken_at: :desc, id: :desc)
|
|
||||||
@wide_groups = @measurements.group_by(&:taken_at)
|
|
||||||
@wide_quantities = @measurements.map(&:quantity).uniq.sort_by(&:name)
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -8,10 +8,6 @@ class QuantitiesController < ApplicationController
|
|||||||
raise AccessForbidden unless current_user.at_least(:active)
|
raise AccessForbidden unless current_user.at_least(:active)
|
||||||
end
|
end
|
||||||
|
|
||||||
before_action only: [:new, :edit, :create, :update] do
|
|
||||||
@user_units = current_user.units.ordered
|
|
||||||
end
|
|
||||||
|
|
||||||
def index
|
def index
|
||||||
@quantities = current_user.quantities.ordered.includes(:parent, :subquantities)
|
@quantities = current_user.quantities.ordered.includes(:parent, :subquantities)
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -86,7 +86,6 @@ module ApplicationHelper
|
|||||||
def initialize(...)
|
def initialize(...)
|
||||||
super(...)
|
super(...)
|
||||||
@default_options.merge!(@options.slice(:form))
|
@default_options.merge!(@options.slice(:form))
|
||||||
@default_html_options.merge!(@options.slice(:form))
|
|
||||||
end
|
end
|
||||||
|
|
||||||
[:text_field, :password_field, :text_area].each do |selector|
|
[:text_field, :password_field, :text_area].each do |selector|
|
||||||
|
|||||||
@@ -9,47 +9,6 @@ function showPage(event) {
|
|||||||
}
|
}
|
||||||
document.addEventListener('turbo:load', showPage)
|
document.addEventListener('turbo:load', showPage)
|
||||||
|
|
||||||
function groupMeasurements() {
|
|
||||||
var tbody = document.getElementById('measurements');
|
|
||||||
if (!tbody) return;
|
|
||||||
var prevTakenAt = null;
|
|
||||||
Array.from(tbody.querySelectorAll('tr[data-taken-at]'))
|
|
||||||
.filter(function(row) { return row.style.display !== 'none' })
|
|
||||||
.forEach(function(row) {
|
|
||||||
var takenAt = row.dataset.takenAt;
|
|
||||||
row.classList.toggle('grouped', takenAt !== null && takenAt === prevTakenAt);
|
|
||||||
prevTakenAt = takenAt;
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
function getMeasurementsView() {
|
|
||||||
return localStorage.getItem('measurements-view') || 'compact';
|
|
||||||
}
|
|
||||||
|
|
||||||
function applyMeasurementsView(view) {
|
|
||||||
document.body.dataset.measurementsView = view;
|
|
||||||
}
|
|
||||||
|
|
||||||
function setMeasurementsView(view) {
|
|
||||||
localStorage.setItem('measurements-view', view);
|
|
||||||
applyMeasurementsView(view);
|
|
||||||
}
|
|
||||||
window.setMeasurementsView = setMeasurementsView
|
|
||||||
|
|
||||||
document.addEventListener('turbo:load', function() {
|
|
||||||
var tbody = document.getElementById('measurements');
|
|
||||||
if (!tbody) return;
|
|
||||||
groupMeasurements();
|
|
||||||
applyMeasurementsView(getMeasurementsView());
|
|
||||||
new MutationObserver(function() {
|
|
||||||
groupMeasurements();
|
|
||||||
}).observe(tbody, {
|
|
||||||
childList: true, subtree: true,
|
|
||||||
attributes: true, attributeFilter: ['style']
|
|
||||||
});
|
|
||||||
})
|
|
||||||
|
|
||||||
function detailsChange(event) {
|
function detailsChange(event) {
|
||||||
var target = event.currentTarget
|
var target = event.currentTarget
|
||||||
var count = target.querySelectorAll('input:checked:not([disabled])').length
|
var count = target.querySelectorAll('input:checked:not([disabled])').length
|
||||||
@@ -78,41 +37,6 @@ window.detailsObserver = new MutationObserver((mutations) => {
|
|||||||
mutations[0].target.dispatchEvent(new Event('change', {bubbles: true}))
|
mutations[0].target.dispatchEvent(new Event('change', {bubbles: true}))
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|
||||||
function readoutUnitChanged(select) {
|
|
||||||
var button = select.closest('tr').querySelector('.set-default-unit');
|
|
||||||
if (select.value && select.value !== select.dataset.defaultUnitId) {
|
|
||||||
Turbo.StreamElement.prototype.enableElement(button);
|
|
||||||
} else {
|
|
||||||
Turbo.StreamElement.prototype.disableElement(button);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
window.readoutUnitChanged = readoutUnitChanged
|
|
||||||
|
|
||||||
function setDefaultUnit(button) {
|
|
||||||
var select = button.closest('tr').querySelector('select[data-default-unit-id]');
|
|
||||||
var form = document.createElement('form');
|
|
||||||
form.action = button.dataset.path;
|
|
||||||
form.method = 'post';
|
|
||||||
form.dataset.turboStream = 'true';
|
|
||||||
var methodInput = document.createElement('input');
|
|
||||||
methodInput.type = 'hidden'; methodInput.name = '_method'; methodInput.value = 'patch';
|
|
||||||
var unitInput = document.createElement('input');
|
|
||||||
unitInput.type = 'hidden'; unitInput.name = 'quantity[default_unit_id]'; unitInput.value = select.value;
|
|
||||||
form.appendChild(methodInput);
|
|
||||||
form.appendChild(unitInput);
|
|
||||||
form.addEventListener('turbo:submit-end', function(event) {
|
|
||||||
if (event.detail.success) {
|
|
||||||
select.dataset.defaultUnitId = select.value;
|
|
||||||
readoutUnitChanged(select);
|
|
||||||
}
|
|
||||||
form.remove();
|
|
||||||
});
|
|
||||||
document.body.appendChild(form);
|
|
||||||
form.requestSubmit();
|
|
||||||
}
|
|
||||||
window.setDefaultUnit = setDefaultUnit
|
|
||||||
|
|
||||||
function formValidate(event) {
|
function formValidate(event) {
|
||||||
var id = event.submitter.getAttribute("data-validate")
|
var id = event.submitter.getAttribute("data-validate")
|
||||||
if (!id) return;
|
if (!id) return;
|
||||||
@@ -282,17 +206,22 @@ window.dragEnd = dragEnd
|
|||||||
|
|
||||||
function drop(event) {
|
function drop(event) {
|
||||||
event.preventDefault()
|
event.preventDefault()
|
||||||
var idParam = event.currentTarget.getAttribute("data-drop-id-param")
|
|
||||||
|
var params = new URLSearchParams()
|
||||||
|
var id_param = event.currentTarget.getAttribute("data-drop-id-param")
|
||||||
var id = event.currentTarget.getAttribute("data-drop-id").split("_").pop()
|
var id = event.currentTarget.getAttribute("data-drop-id").split("_").pop()
|
||||||
var form = document.createElement('form');
|
params.append(id_param, id)
|
||||||
form.action = event.dataTransfer.getData("text/plain");
|
|
||||||
form.method = 'post';
|
fetch(event.dataTransfer.getData("text/plain"), {
|
||||||
form.dataset.turboStream = 'true';
|
body: params,
|
||||||
var input = document.createElement('input');
|
headers: {
|
||||||
input.type = 'hidden'; input.name = idParam; input.value = id;
|
"Accept": "text/vnd.turbo-stream.html",
|
||||||
form.appendChild(input);
|
"X-CSRF-Token": document.head.querySelector("meta[name=csrf-token]").content,
|
||||||
form.addEventListener('turbo:submit-end', function() { form.remove(); });
|
"X-Requested-With": "XMLHttpRequest"
|
||||||
document.body.appendChild(form);
|
},
|
||||||
form.requestSubmit();
|
method: "POST"
|
||||||
|
})
|
||||||
|
.then(response => response.text())
|
||||||
|
.then(html => Turbo.renderStreamMessage(html))
|
||||||
}
|
}
|
||||||
window.drop = drop
|
window.drop = drop
|
||||||
|
|||||||
@@ -1,10 +1,9 @@
|
|||||||
class Quantity < ApplicationRecord
|
class Quantity < ApplicationRecord
|
||||||
ATTRIBUTES = [:name, :description, :parent_id, :default_unit_id]
|
ATTRIBUTES = [:name, :description, :parent_id]
|
||||||
attr_cached :depth, :pathname
|
attr_cached :depth, :pathname
|
||||||
|
|
||||||
belongs_to :user, optional: true
|
belongs_to :user, optional: true
|
||||||
belongs_to :parent, optional: true, class_name: "Quantity"
|
belongs_to :parent, optional: true, class_name: "Quantity"
|
||||||
belongs_to :default_unit, optional: true, class_name: "Unit"
|
|
||||||
has_many :subquantities, ->{ order(:name) }, class_name: "Quantity",
|
has_many :subquantities, ->{ order(:name) }, class_name: "Quantity",
|
||||||
inverse_of: :parent, dependent: :restrict_with_error
|
inverse_of: :parent, dependent: :restrict_with_error
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
class Readout < ApplicationRecord
|
class Readout < ApplicationRecord
|
||||||
ATTRIBUTES = [:quantity_id, :value, :unit_id, :taken_at]
|
ATTRIBUTES = [:quantity_id, :value, :unit_id]
|
||||||
|
|
||||||
belongs_to :user
|
belongs_to :user
|
||||||
belongs_to :quantity
|
belongs_to :quantity
|
||||||
|
|||||||
@@ -1,24 +0,0 @@
|
|||||||
<%= tabular_fields_for @readout, form: form_tag do |form| %>
|
|
||||||
<%- tag.tr id: row, class: "form", onkeydown: "formProcessKey(event)",
|
|
||||||
data: {form: form_tag, hidden_row: hidden_row, link: link} do %>
|
|
||||||
<td><%= @readout.quantity %></td>
|
|
||||||
<td class="ralign">
|
|
||||||
<%= form.number_field :value, required: true, autofocus: true %>
|
|
||||||
</td>
|
|
||||||
<td>
|
|
||||||
<%= form.collection_select :unit_id, @user_units, :id,
|
|
||||||
->(u){ sanitize(' ' * (u.base_id? ? 1 : 0) + u.symbol) },
|
|
||||||
{}, required: true %>
|
|
||||||
</td>
|
|
||||||
<td>
|
|
||||||
<%= form.datetime_field :taken_at %>
|
|
||||||
</td>
|
|
||||||
<td></td>
|
|
||||||
<td class="flex">
|
|
||||||
<%= form.button %>
|
|
||||||
<%= image_link_to t(:cancel), "close-outline", measurements_path,
|
|
||||||
class: 'dangerous', name: :cancel,
|
|
||||||
onclick: render_turbo_stream('edit_form_close', {row: row}) %>
|
|
||||||
</td>
|
|
||||||
<% end %>
|
|
||||||
<% end %>
|
|
||||||
@@ -1,2 +0,0 @@
|
|||||||
<%= turbo_stream.close_form row %>
|
|
||||||
<%= turbo_stream.update :flashes %>
|
|
||||||
@@ -1,33 +0,0 @@
|
|||||||
<% form_tag = dom_id(@readout, :edit, :form) %>
|
|
||||||
<% row = dom_id(@readout, :edit) %>
|
|
||||||
<% hidden_row = dom_id(@readout) %>
|
|
||||||
|
|
||||||
<%= tabular_form_with model: @readout, url: measurement_path(@readout),
|
|
||||||
id: form_tag do |form| %>
|
|
||||||
<table class="items-table">
|
|
||||||
<tbody>
|
|
||||||
<%= tag.tr id: row, class: "form", onkeydown: "formProcessKey(event)",
|
|
||||||
data: {form: form_tag, hidden_row: hidden_row} do %>
|
|
||||||
<td><%= @readout.quantity %></td>
|
|
||||||
<td class="ralign">
|
|
||||||
<%= form.number_field :value, required: true, autofocus: true %>
|
|
||||||
</td>
|
|
||||||
<td>
|
|
||||||
<%= form.collection_select :unit_id, @user_units, :id,
|
|
||||||
->(u){ sanitize(' ' * (u.base_id? ? 1 : 0) + u.symbol) },
|
|
||||||
{}, required: true %>
|
|
||||||
</td>
|
|
||||||
<td>
|
|
||||||
<%= form.datetime_field :taken_at %>
|
|
||||||
</td>
|
|
||||||
<td></td>
|
|
||||||
<td class="flex">
|
|
||||||
<%= form.button %>
|
|
||||||
<%= image_link_to t(:cancel), "close-outline", measurements_path,
|
|
||||||
class: 'dangerous', name: :cancel,
|
|
||||||
onclick: render_turbo_stream('edit_form_close', {row: row}) %>
|
|
||||||
</td>
|
|
||||||
<% end %>
|
|
||||||
</tbody>
|
|
||||||
</table>
|
|
||||||
<% end %>
|
|
||||||
@@ -8,7 +8,7 @@
|
|||||||
<tr class="italic">
|
<tr class="italic">
|
||||||
<td class="hexpand hmin50"><%= t '.taken_at_html' %></td>
|
<td class="hexpand hmin50"><%= t '.taken_at_html' %></td>
|
||||||
<td colspan="3" class="ralign">
|
<td colspan="3" class="ralign">
|
||||||
<%= form.datetime_field :taken_at, required: true, value: Time.current.strftime('%Y-%m-%dT%H:%M') %>
|
<%= form.datetime_field :taken_at, required: true %>
|
||||||
</td>
|
</td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
@@ -1,22 +0,0 @@
|
|||||||
<%= tag.tr id: dom_id(readout), data: {taken_at: readout.taken_at&.iso8601,
|
|
||||||
quantity_id: readout.quantity_id, quantity_name: readout.quantity.name,
|
|
||||||
value: format("%.10g", readout.value), unit: readout.unit.symbol} do %>
|
|
||||||
<td>
|
|
||||||
<% if current_user.at_least(:active) %>
|
|
||||||
<%= link_to readout.quantity, edit_measurement_path(readout),
|
|
||||||
class: 'link', onclick: 'this.blur();', data: {turbo_stream: true} %>
|
|
||||||
<% else %>
|
|
||||||
<%= readout.quantity %>
|
|
||||||
<% end %>
|
|
||||||
</td>
|
|
||||||
<td class="ralign"><%= format("%.10g", readout.value) %></td>
|
|
||||||
<td><%= readout.unit %></td>
|
|
||||||
<td class="taken-at"><%= l(readout.taken_at) if readout.taken_at %></td>
|
|
||||||
<td class="created-at"><%= l(readout.created_at) %></td>
|
|
||||||
<% if current_user.at_least(:active) %>
|
|
||||||
<td class="flex">
|
|
||||||
<%= image_button_to t('.destroy'), 'delete-outline', measurement_path(readout),
|
|
||||||
method: :delete, data: {turbo_stream: true} %>
|
|
||||||
</td>
|
|
||||||
<% end %>
|
|
||||||
<% end %>
|
|
||||||
@@ -1,41 +0,0 @@
|
|||||||
<table class="items-table">
|
|
||||||
<thead>
|
|
||||||
<tr>
|
|
||||||
<th><%= Readout.human_attribute_name(:taken_at) %></th>
|
|
||||||
<% wide_quantities.each do |q| %>
|
|
||||||
<th><%= q.name %></th>
|
|
||||||
<% end %>
|
|
||||||
<th><%= Readout.human_attribute_name(:created_at) %></th>
|
|
||||||
</tr>
|
|
||||||
</thead>
|
|
||||||
<tbody>
|
|
||||||
<% wide_groups.each do |taken_at, readouts| %>
|
|
||||||
<tr>
|
|
||||||
<td><%= l(taken_at) if taken_at %></td>
|
|
||||||
<% wide_quantities.each do |q| %>
|
|
||||||
<% readout = readouts.find { |r| r.quantity_id == q.id } %>
|
|
||||||
<td class="ralign">
|
|
||||||
<% if readout %>
|
|
||||||
<span class="wide-cell">
|
|
||||||
<% if current_user.at_least(:active) %>
|
|
||||||
<%= link_to format("%.10g", readout.value),
|
|
||||||
edit_measurement_path(readout, view: :wide),
|
|
||||||
class: 'link', onclick: 'this.blur();',
|
|
||||||
data: {turbo_stream: true} %>
|
|
||||||
<% else %>
|
|
||||||
<%= format("%.10g", readout.value) %>
|
|
||||||
<% end %>
|
|
||||||
<%= readout.unit.symbol %>
|
|
||||||
<% if current_user.at_least(:active) %>
|
|
||||||
<%= image_button_to '', 'delete-outline', measurement_path(readout),
|
|
||||||
method: :delete, data: {turbo_stream: true} %>
|
|
||||||
<% end %>
|
|
||||||
</span>
|
|
||||||
<% end %>
|
|
||||||
</td>
|
|
||||||
<% end %>
|
|
||||||
<td><%= l(readouts.first.created_at) %></td>
|
|
||||||
</tr>
|
|
||||||
<% end %>
|
|
||||||
</tbody>
|
|
||||||
</table>
|
|
||||||
@@ -1,13 +0,0 @@
|
|||||||
<% if @readouts.present? && @readouts.all?(&:persisted?) %>
|
|
||||||
<%= turbo_stream.update :flashes %>
|
|
||||||
<%= turbo_stream.remove :measurement_form %>
|
|
||||||
<%= turbo_stream.enable :new_measurement_link %>
|
|
||||||
<%= turbo_stream.remove :no_items %>
|
|
||||||
<% @readouts.each do |readout| %>
|
|
||||||
<%= turbo_stream.prepend :measurements, partial: 'readout', locals: {readout: readout} %>
|
|
||||||
<% end %>
|
|
||||||
<%= turbo_stream.update 'measurements-wide', partial: 'wide_table',
|
|
||||||
locals: {wide_groups: @wide_groups, wide_quantities: @wide_quantities} %>
|
|
||||||
<% else %>
|
|
||||||
<%= turbo_stream.update :flashes %>
|
|
||||||
<% end %>
|
|
||||||
@@ -1,5 +0,0 @@
|
|||||||
<%= turbo_stream.update :flashes %>
|
|
||||||
<%= turbo_stream.remove @readout %>
|
|
||||||
<%= turbo_stream.append(:measurements, render_no_items) if current_user.readouts.empty? %>
|
|
||||||
<%= turbo_stream.update 'measurements-wide', partial: 'wide_table',
|
|
||||||
locals: {wide_groups: @wide_groups, wide_quantities: @wide_quantities} %>
|
|
||||||
@@ -1,18 +0,0 @@
|
|||||||
<% ids = {row: dom_id(@readout, :edit),
|
|
||||||
hidden_row: dom_id(@readout),
|
|
||||||
link: nil,
|
|
||||||
form_tag: dom_id(@readout, :edit, :form)} %>
|
|
||||||
|
|
||||||
<% if params[:view] == 'wide' %>
|
|
||||||
<%= turbo_stream.update :measurement_edit_form, partial: 'edit_panel' %>
|
|
||||||
<%= turbo_stream.hide ids[:hidden_row] %>
|
|
||||||
<% else %>
|
|
||||||
<%= turbo_stream.append :measurement_edit_form do %>
|
|
||||||
<%- tabular_form_with model: @readout, url: measurement_path(@readout),
|
|
||||||
html: {id: ids[:form_tag]} do %>
|
|
||||||
<% end %>
|
|
||||||
<% end %>
|
|
||||||
<%= turbo_stream.hide ids[:hidden_row] %>
|
|
||||||
<%= turbo_stream.remove ids[:row] %>
|
|
||||||
<%= turbo_stream.after @readout, partial: 'edit_form', locals: ids -%>
|
|
||||||
<% end %>
|
|
||||||
@@ -5,36 +5,10 @@
|
|||||||
id: :new_measurement_link, onclick: 'this.blur();',
|
id: :new_measurement_link, onclick: 'this.blur();',
|
||||||
data: {turbo_stream: true} %>
|
data: {turbo_stream: true} %>
|
||||||
<% end %>
|
<% end %>
|
||||||
<%= image_button_tag '', 'view-rows', name: nil, type: 'button',
|
|
||||||
class: 'view-toggle', title: t('.view_compact'),
|
|
||||||
data: {view: 'compact'}, onclick: "setMeasurementsView('compact')" %>
|
|
||||||
<%= image_button_tag '', 'view-columns', name: nil, type: 'button',
|
|
||||||
class: 'view-toggle', title: t('.view_wide'),
|
|
||||||
data: {view: 'wide'}, onclick: "setMeasurementsView('wide')" %>
|
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="main-area measurements-section">
|
<table class="main-area">
|
||||||
<%= tag.div id: :measurement_edit_form %>
|
|
||||||
<table class="items-table measurements-compact">
|
|
||||||
<thead>
|
|
||||||
<tr>
|
|
||||||
<th><%= Quantity.model_name.human %></th>
|
|
||||||
<th><%= Readout.human_attribute_name(:value) %></th>
|
|
||||||
<th><%= Unit.model_name.human %></th>
|
|
||||||
<th data-column="taken-at"><%= Readout.human_attribute_name(:taken_at) %></th>
|
|
||||||
<th data-column="created-at"><%= Readout.human_attribute_name(:created_at) %></th>
|
|
||||||
<% if current_user.at_least(:active) %>
|
|
||||||
<th></th>
|
|
||||||
<% end %>
|
|
||||||
</tr>
|
|
||||||
</thead>
|
|
||||||
<tbody id="measurements">
|
<tbody id="measurements">
|
||||||
<%= render(partial: 'readout', collection: @measurements, as: :readout) || render_no_items %>
|
<%= render(@measurements) || render_no_items %>
|
||||||
</tbody>
|
</tbody>
|
||||||
</table>
|
</table>
|
||||||
|
|
||||||
<div id="measurements-wide" class="measurements-wide">
|
|
||||||
<%= render 'wide_table', wide_groups: @wide_groups, wide_quantities: @wide_quantities %>
|
|
||||||
</div>
|
|
||||||
</div>
|
|
||||||
|
|
||||||
|
|||||||
@@ -1,4 +0,0 @@
|
|||||||
<%= turbo_stream.close_form dom_id(@readout, :edit) %>
|
|
||||||
<%= turbo_stream.replace @readout, partial: 'measurements/readout', locals: {readout: @readout} %>
|
|
||||||
<%= turbo_stream.update 'measurements-wide', partial: 'wide_table',
|
|
||||||
locals: {wide_groups: @wide_groups, wide_quantities: @wide_quantities} %>
|
|
||||||
@@ -8,11 +8,6 @@
|
|||||||
<td>
|
<td>
|
||||||
<%= form.text_area :description, cols: 30, rows: 1, escape: false %>
|
<%= form.text_area :description, cols: 30, rows: 1, escape: false %>
|
||||||
</td>
|
</td>
|
||||||
<td>
|
|
||||||
<%= form.collection_select :default_unit_id, @user_units, :id,
|
|
||||||
->(u){ sanitize(' ' * (u.base_id? ? 1 : 0) + u.symbol) },
|
|
||||||
{include_blank: true}, onchange: "this.dataset.changed = ''" %>
|
|
||||||
</td>
|
|
||||||
|
|
||||||
<td class="flex">
|
<td class="flex">
|
||||||
<%= form.button %>
|
<%= form.button %>
|
||||||
|
|||||||
@@ -10,7 +10,6 @@
|
|||||||
onclick: 'this.blur();', data: {turbo_stream: true} %>
|
onclick: 'this.blur();', data: {turbo_stream: true} %>
|
||||||
</td>
|
</td>
|
||||||
<td><%= quantity.description %></td>
|
<td><%= quantity.description %></td>
|
||||||
<td><%= quantity.default_unit&.symbol %></td>
|
|
||||||
|
|
||||||
<% if current_user.at_least(:active) %>
|
<% if current_user.at_least(:active) %>
|
||||||
<td class="flex">
|
<td class="flex">
|
||||||
|
|||||||
@@ -16,7 +16,6 @@
|
|||||||
<tr>
|
<tr>
|
||||||
<th><%= Quantity.human_attribute_name(:name) %></th>
|
<th><%= Quantity.human_attribute_name(:name) %></th>
|
||||||
<th class="hexpand"><%= Quantity.human_attribute_name(:description) %></th>
|
<th class="hexpand"><%= Quantity.human_attribute_name(:description) %></th>
|
||||||
<th><%= Quantity.human_attribute_name(:default_unit) %></th>
|
|
||||||
<% if current_user.at_least(:active) %>
|
<% if current_user.at_least(:active) %>
|
||||||
<th><%= t :actions %></th>
|
<th><%= t :actions %></th>
|
||||||
<th></th>
|
<th></th>
|
||||||
@@ -26,7 +25,7 @@
|
|||||||
ondragover: "dragOver(event)", ondrop: "drop(event)",
|
ondragover: "dragOver(event)", ondrop: "drop(event)",
|
||||||
ondragenter: "dragEnter(event)", ondragleave: "dragLeave(event)",
|
ondragenter: "dragEnter(event)", ondragleave: "dragLeave(event)",
|
||||||
data: {drop_id: "quantity_", drop_id_param: "quantity[parent_id]"} do %>
|
data: {drop_id: "quantity_", drop_id_param: "quantity[parent_id]"} do %>
|
||||||
<th colspan="5"><%= t '.top_level_drop' %></th>
|
<th colspan="4"><%= t '.top_level_drop' %></th>
|
||||||
<% end %>
|
<% end %>
|
||||||
</thead>
|
</thead>
|
||||||
<tbody id="quantities">
|
<tbody id="quantities">
|
||||||
|
|||||||
@@ -12,17 +12,10 @@
|
|||||||
<td>
|
<td>
|
||||||
<%= form.collection_select :unit_id, @user_units, :id,
|
<%= form.collection_select :unit_id, @user_units, :id,
|
||||||
->(u){ sanitize(' ' * (u.base_id ? 1 : 0) + u.symbol) },
|
->(u){ sanitize(' ' * (u.base_id ? 1 : 0) + u.symbol) },
|
||||||
{prompt: '', disabled: '', selected: readout.quantity.default_unit_id || ''}, required: true,
|
{prompt: '', disabled: '', selected: ''}, required: true %>
|
||||||
data: {default_unit_id: readout.quantity.default_unit_id || ''},
|
|
||||||
onchange: "readoutUnitChanged(this)" %>
|
|
||||||
</td>
|
</td>
|
||||||
<td class="flex">
|
<td class="flex">
|
||||||
<%# TODO: change to _link_ after giving up displaying relative paths %>
|
<%# TODO: change to _link_ after giving up displaying relative paths %>
|
||||||
<%= image_button_tag '', 'check-circle-outline',
|
|
||||||
class: 'set-default-unit', name: nil, type: 'button', disabled: true,
|
|
||||||
title: t('readouts.form.set_default_unit'),
|
|
||||||
data: {path: quantity_path(readout.quantity)},
|
|
||||||
onclick: 'setDefaultUnit(this)' %>
|
|
||||||
<%= image_button_tag '', 'delete-outline', class: 'dangerous', name: nil,
|
<%= image_button_tag '', 'delete-outline', class: 'dangerous', name: nil,
|
||||||
formaction: discard_readouts_path(readout.quantity),
|
formaction: discard_readouts_path(readout.quantity),
|
||||||
formmethod: :get, formnovalidate: true, data: {turbo_stream: true} %>
|
formmethod: :get, formnovalidate: true, data: {turbo_stream: true} %>
|
||||||
|
|||||||
@@ -20,6 +20,10 @@ Bundler.require(*Rails.groups)
|
|||||||
|
|
||||||
module FixinMe
|
module FixinMe
|
||||||
class Application < Rails::Application
|
class Application < Rails::Application
|
||||||
|
# Allow RAILS_DATABASE_YML to override the database config file path.
|
||||||
|
# Used by `rails test:all_databases` to test against multiple DB adapters.
|
||||||
|
config.paths['config/database'] = [ENV['RAILS_DATABASE_YML']] if ENV['RAILS_DATABASE_YML']
|
||||||
|
|
||||||
# Initialize configuration defaults for originally generated Rails version.
|
# Initialize configuration defaults for originally generated Rails version.
|
||||||
config.load_defaults 7.0
|
config.load_defaults 7.0
|
||||||
|
|
||||||
|
|||||||
@@ -48,3 +48,26 @@ production:
|
|||||||
#test:
|
#test:
|
||||||
# <<: *default
|
# <<: *default
|
||||||
# database: fixinme_test
|
# database: fixinme_test
|
||||||
|
|
||||||
|
# Multi-database testing — `bundle exec rails test:all_databases`
|
||||||
|
# ---------------------------------------------------------------
|
||||||
|
# Add any number of `test_<name>:` blocks to run the full test suite
|
||||||
|
# against additional database adapters in a single command.
|
||||||
|
# Each adapter's gem must be available in the bundle:
|
||||||
|
# bundle config --local with "mysql:sqlite" # mysql + sqlite
|
||||||
|
# bundle config --local with "mysql:pg" # mysql + postgresql
|
||||||
|
#
|
||||||
|
# Example — run tests against MySQL and SQLite:
|
||||||
|
#
|
||||||
|
#test_sqlite:
|
||||||
|
# adapter: sqlite3
|
||||||
|
# database: db/fixinme_test.sqlite3
|
||||||
|
#
|
||||||
|
# Example — run tests against MySQL and PostgreSQL:
|
||||||
|
#
|
||||||
|
#test_pg:
|
||||||
|
# adapter: postgresql
|
||||||
|
# database: fixinme_test
|
||||||
|
# username: fixinme
|
||||||
|
# password: Some-password1%
|
||||||
|
# host: localhost
|
||||||
|
|||||||
@@ -58,7 +58,4 @@ Rails.application.configure do
|
|||||||
# config.action_view.annotate_rendered_view_with_filenames = true
|
# config.action_view.annotate_rendered_view_with_filenames = true
|
||||||
|
|
||||||
config.log_level = :info
|
config.log_level = :info
|
||||||
|
|
||||||
# Allow Capybara's dynamic test server host (127.0.0.1:<random_port>)
|
|
||||||
config.hosts << '127.0.0.1'
|
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -11,13 +11,8 @@ en:
|
|||||||
activerecord:
|
activerecord:
|
||||||
attributes:
|
attributes:
|
||||||
quantity:
|
quantity:
|
||||||
default_unit: Default unit
|
|
||||||
description: Description
|
description: Description
|
||||||
name: Name
|
name: Name
|
||||||
readout:
|
|
||||||
created_at: Recorded at
|
|
||||||
taken_at: Taken at
|
|
||||||
value: Value
|
|
||||||
unit:
|
unit:
|
||||||
base: Base unit
|
base: Base unit
|
||||||
description: Description
|
description: Description
|
||||||
@@ -86,9 +81,6 @@ en:
|
|||||||
revert: Revert
|
revert: Revert
|
||||||
sign_out: Sign out
|
sign_out: Sign out
|
||||||
source_code: Get code
|
source_code: Get code
|
||||||
readouts:
|
|
||||||
form:
|
|
||||||
set_default_unit: Set as default unit
|
|
||||||
measurements:
|
measurements:
|
||||||
navigation: Measurements
|
navigation: Measurements
|
||||||
no_items: There are no measurements taken. You can Add some now.
|
no_items: There are no measurements taken. You can Add some now.
|
||||||
@@ -97,20 +89,6 @@ en:
|
|||||||
taken_at_html: Measurement taken at 
|
taken_at_html: Measurement taken at 
|
||||||
index:
|
index:
|
||||||
new_measurement: Add measurement
|
new_measurement: Add measurement
|
||||||
view_compact: Compact view
|
|
||||||
view_wide: Wide view
|
|
||||||
readout:
|
|
||||||
edit: Edit
|
|
||||||
destroy: Delete
|
|
||||||
create:
|
|
||||||
success:
|
|
||||||
one: Recorded 1 measurement.
|
|
||||||
other: Recorded %{count} measurements.
|
|
||||||
no_readouts: No readouts selected.
|
|
||||||
update:
|
|
||||||
success: Measurement updated.
|
|
||||||
destroy:
|
|
||||||
success: Measurement deleted.
|
|
||||||
quantities:
|
quantities:
|
||||||
navigation: Quantities
|
navigation: Quantities
|
||||||
no_items: There are no configured quantities. You can Add some or Import from defaults.
|
no_items: There are no configured quantities. You can Add some or Import from defaults.
|
||||||
|
|||||||
@@ -1,5 +0,0 @@
|
|||||||
class AddTakenAtToReadouts < ActiveRecord::Migration[7.2]
|
|
||||||
def change
|
|
||||||
add_column :readouts, :taken_at, :datetime
|
|
||||||
end
|
|
||||||
end
|
|
||||||
@@ -1,5 +0,0 @@
|
|||||||
class AddDefaultUnitToQuantities < ActiveRecord::Migration[7.2]
|
|
||||||
def change
|
|
||||||
add_reference :quantities, :default_unit, foreign_key: {to_table: :units}, null: true
|
|
||||||
end
|
|
||||||
end
|
|
||||||
@@ -10,7 +10,7 @@
|
|||||||
#
|
#
|
||||||
# It's strongly recommended that you check this file into your version control system.
|
# It's strongly recommended that you check this file into your version control system.
|
||||||
|
|
||||||
ActiveRecord::Schema[7.2].define(version: 2026_04_03_000000) do
|
ActiveRecord::Schema[7.2].define(version: 2025_01_21_230456) do
|
||||||
create_table "quantities", charset: "utf8mb4", collation: "utf8mb4_0900_as_ci", force: :cascade do |t|
|
create_table "quantities", charset: "utf8mb4", collation: "utf8mb4_0900_as_ci", force: :cascade do |t|
|
||||||
t.bigint "user_id"
|
t.bigint "user_id"
|
||||||
t.string "name", limit: 31, null: false
|
t.string "name", limit: 31, null: false
|
||||||
@@ -20,8 +20,6 @@ ActiveRecord::Schema[7.2].define(version: 2026_04_03_000000) do
|
|||||||
t.datetime "updated_at", null: false
|
t.datetime "updated_at", null: false
|
||||||
t.integer "depth", default: 0, null: false
|
t.integer "depth", default: 0, null: false
|
||||||
t.string "pathname", limit: 511, null: false
|
t.string "pathname", limit: 511, null: false
|
||||||
t.bigint "default_unit_id"
|
|
||||||
t.index ["default_unit_id"], name: "index_quantities_on_default_unit_id"
|
|
||||||
t.index ["parent_id"], name: "index_quantities_on_parent_id"
|
t.index ["parent_id"], name: "index_quantities_on_parent_id"
|
||||||
t.index ["user_id", "parent_id", "name"], name: "index_quantities_on_user_id_and_parent_id_and_name", unique: true
|
t.index ["user_id", "parent_id", "name"], name: "index_quantities_on_user_id_and_parent_id_and_name", unique: true
|
||||||
t.index ["user_id"], name: "index_quantities_on_user_id"
|
t.index ["user_id"], name: "index_quantities_on_user_id"
|
||||||
@@ -34,7 +32,6 @@ ActiveRecord::Schema[7.2].define(version: 2026_04_03_000000) do
|
|||||||
t.decimal "value", precision: 30, scale: 15, null: false
|
t.decimal "value", precision: 30, scale: 15, null: false
|
||||||
t.datetime "created_at", null: false
|
t.datetime "created_at", null: false
|
||||||
t.datetime "updated_at", null: false
|
t.datetime "updated_at", null: false
|
||||||
t.datetime "taken_at"
|
|
||||||
t.index ["quantity_id", "created_at"], name: "index_readouts_on_quantity_id_and_created_at", unique: true
|
t.index ["quantity_id", "created_at"], name: "index_readouts_on_quantity_id_and_created_at", unique: true
|
||||||
t.index ["quantity_id"], name: "index_readouts_on_quantity_id"
|
t.index ["quantity_id"], name: "index_readouts_on_quantity_id"
|
||||||
t.index ["unit_id"], name: "index_readouts_on_unit_id"
|
t.index ["unit_id"], name: "index_readouts_on_unit_id"
|
||||||
@@ -73,7 +70,6 @@ ActiveRecord::Schema[7.2].define(version: 2026_04_03_000000) do
|
|||||||
end
|
end
|
||||||
|
|
||||||
add_foreign_key "quantities", "quantities", column: "parent_id", on_delete: :cascade
|
add_foreign_key "quantities", "quantities", column: "parent_id", on_delete: :cascade
|
||||||
add_foreign_key "quantities", "units", column: "default_unit_id"
|
|
||||||
add_foreign_key "quantities", "users"
|
add_foreign_key "quantities", "users"
|
||||||
add_foreign_key "readouts", "quantities"
|
add_foreign_key "readouts", "quantities"
|
||||||
add_foreign_key "readouts", "units"
|
add_foreign_key "readouts", "units"
|
||||||
|
|||||||
118
lib/tasks/test_databases.rake
Normal file
118
lib/tasks/test_databases.rake
Normal file
@@ -0,0 +1,118 @@
|
|||||||
|
require 'yaml'
|
||||||
|
require 'erb'
|
||||||
|
require 'tmpdir'
|
||||||
|
|
||||||
|
namespace :test do
|
||||||
|
desc <<~DESC
|
||||||
|
Run the full test suite against every test database configured in database.yml.
|
||||||
|
|
||||||
|
Any top-level key that starts with "test" and contains a Hash is treated as a
|
||||||
|
test database configuration:
|
||||||
|
|
||||||
|
test: # always required — the primary test database
|
||||||
|
adapter: mysql2
|
||||||
|
database: fixinme_test
|
||||||
|
...
|
||||||
|
|
||||||
|
test_sqlite: # optional additional databases
|
||||||
|
adapter: sqlite3
|
||||||
|
database: db/fixinme_test.sqlite3
|
||||||
|
|
||||||
|
test_pg:
|
||||||
|
adapter: postgresql
|
||||||
|
database: fixinme_test
|
||||||
|
...
|
||||||
|
|
||||||
|
For each database the task will:
|
||||||
|
1. Check that the required adapter gem is available (skip with warning if not).
|
||||||
|
2. Run `rails db:test:prepare` to create/migrate the database.
|
||||||
|
3. Run `rails test` and record pass/fail.
|
||||||
|
|
||||||
|
A summary is printed at the end. The task exits non-zero if any database fails.
|
||||||
|
DESC
|
||||||
|
task :all_databases do
|
||||||
|
db_file = Rails.root.join('config', 'database.yml')
|
||||||
|
all = YAML.safe_load(ERB.new(db_file.read).result, aliases: true) || {}
|
||||||
|
test_cfgs = all.select { |k, v| k.to_s.start_with?('test') && v.is_a?(Hash) }
|
||||||
|
non_test = all.reject { |k, _| k.to_s.start_with?('test') }
|
||||||
|
|
||||||
|
abort "No test database configurations found in #{db_file}." if test_cfgs.empty?
|
||||||
|
|
||||||
|
results = {}
|
||||||
|
|
||||||
|
test_cfgs.each do |name, config|
|
||||||
|
adapter = config['adapter'].to_s
|
||||||
|
|
||||||
|
puts "\n#{'─' * 64}"
|
||||||
|
puts " #{name} (adapter: #{adapter})"
|
||||||
|
puts '─' * 64
|
||||||
|
|
||||||
|
unless adapter_available?(adapter)
|
||||||
|
puts " SKIPPED — gem for '#{adapter}' adapter not available in bundle."
|
||||||
|
puts " Add it to Gemfile (e.g. `bundle config --local with #{adapter_group(adapter)}`)"
|
||||||
|
results[name] = :skipped
|
||||||
|
next
|
||||||
|
end
|
||||||
|
|
||||||
|
Dir.mktmpdir('rails_test_db_') do |tmpdir|
|
||||||
|
tmp_yml = File.join(tmpdir, 'database.yml')
|
||||||
|
# Write a standalone database.yml with just this config as `test:`
|
||||||
|
# (non-test configs are preserved so boot doesn't fail on `production:` lookup)
|
||||||
|
File.write(tmp_yml, non_test.merge('test' => config).to_yaml)
|
||||||
|
|
||||||
|
env = { 'RAILS_DATABASE_YML' => tmp_yml }
|
||||||
|
|
||||||
|
if system(env, 'bundle exec rails db:test:prepare')
|
||||||
|
results[name] = system(env, 'bundle exec rails test') ? :pass : :fail
|
||||||
|
else
|
||||||
|
results[name] = :prepare_failed
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
puts "\n#{'═' * 64}"
|
||||||
|
puts " SUMMARY"
|
||||||
|
puts '═' * 64
|
||||||
|
results.each do |name, status|
|
||||||
|
adapter = test_cfgs[name]['adapter']
|
||||||
|
icon = { pass: '✓', fail: '✗', prepare_failed: '✗', skipped: '–' }[status]
|
||||||
|
label = { pass: 'PASS', fail: 'FAIL',
|
||||||
|
prepare_failed: 'PREPARE FAILED', skipped: 'SKIPPED' }[status]
|
||||||
|
puts " #{icon} #{name.ljust(26)} (#{adapter.ljust(12)}) #{label}"
|
||||||
|
end
|
||||||
|
puts '═' * 64
|
||||||
|
|
||||||
|
failed = results.count { |_, s| s == :fail || s == :prepare_failed }
|
||||||
|
abort "\n #{failed} database(s) failed." if failed > 0
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
private
|
||||||
|
|
||||||
|
# Returns true if the gem required for this adapter is loadable.
|
||||||
|
ADAPTER_GEMS = {
|
||||||
|
'mysql2' => 'mysql2',
|
||||||
|
'sqlite3' => 'sqlite3',
|
||||||
|
'postgresql' => 'pg',
|
||||||
|
'pg' => 'pg',
|
||||||
|
}.freeze
|
||||||
|
|
||||||
|
def adapter_available?(adapter)
|
||||||
|
gem_name = ADAPTER_GEMS.fetch(adapter, adapter)
|
||||||
|
require gem_name
|
||||||
|
true
|
||||||
|
rescue LoadError
|
||||||
|
false
|
||||||
|
end
|
||||||
|
|
||||||
|
# Returns the Bundler group name that installs the adapter gem.
|
||||||
|
ADAPTER_GROUPS = {
|
||||||
|
'mysql2' => 'mysql',
|
||||||
|
'sqlite3' => 'sqlite',
|
||||||
|
'postgresql' => 'postgresql',
|
||||||
|
'pg' => 'postgresql',
|
||||||
|
}.freeze
|
||||||
|
|
||||||
|
def adapter_group(adapter)
|
||||||
|
ADAPTER_GROUPS.fetch(adapter, adapter)
|
||||||
|
end
|
||||||
@@ -1,65 +1,8 @@
|
|||||||
require "test_helper"
|
require "test_helper"
|
||||||
|
|
||||||
class MeasurementsControllerTest < ActionDispatch::IntegrationTest
|
class MeasurementsControllerTest < ActionDispatch::IntegrationTest
|
||||||
setup do
|
#test "should get index" do
|
||||||
host! '127.0.0.1'
|
# get measurements_index_url
|
||||||
@user = users(:alice)
|
# assert_response :success
|
||||||
post new_user_session_path, params: { user: { email: @user.email, password: 'alice' } }
|
#end
|
||||||
@quantity = @user.quantities.create!(name: 'Weight')
|
|
||||||
@unit = @user.units.create!(symbol: 'kg')
|
|
||||||
end
|
|
||||||
|
|
||||||
test "index returns ok" do
|
|
||||||
get measurements_path
|
|
||||||
assert_response :success
|
|
||||||
end
|
|
||||||
|
|
||||||
test "index requires authentication" do
|
|
||||||
delete destroy_user_session_path
|
|
||||||
get measurements_path
|
|
||||||
assert_response :redirect
|
|
||||||
end
|
|
||||||
|
|
||||||
test "create records readout with taken_at" do
|
|
||||||
taken_at = 1.day.ago.change(usec: 0)
|
|
||||||
assert_difference -> { @user.readouts.count } do
|
|
||||||
post measurements_path, params: {
|
|
||||||
taken_at: taken_at.iso8601,
|
|
||||||
readouts: [{ quantity_id: @quantity.id, value: '82.5', unit_id: @unit.id }]
|
|
||||||
}, as: :turbo_stream
|
|
||||||
end
|
|
||||||
assert_response :success
|
|
||||||
assert_equal taken_at, @user.readouts.last.taken_at
|
|
||||||
end
|
|
||||||
|
|
||||||
test "create with no readouts selected shows alert" do
|
|
||||||
post measurements_path, params: { taken_at: Time.now.iso8601 }, as: :turbo_stream
|
|
||||||
assert_response :success
|
|
||||||
end
|
|
||||||
|
|
||||||
test "destroy removes readout" do
|
|
||||||
readout = @user.readouts.create!(quantity: @quantity, unit: @unit, value: 82.5, taken_at: 1.day.ago)
|
|
||||||
assert_difference -> { @user.readouts.count }, -1 do
|
|
||||||
delete measurement_path(readout), as: :turbo_stream
|
|
||||||
end
|
|
||||||
assert_response :success
|
|
||||||
end
|
|
||||||
|
|
||||||
test "destroy cannot remove another user's readout" do
|
|
||||||
other_quantity = users(:bob).quantities.create!(name: 'Weight')
|
|
||||||
other_unit = users(:bob).units.create!(symbol: 'kg')
|
|
||||||
readout = users(:bob).readouts.create!(quantity: other_quantity, unit: other_unit, value: 70.0, taken_at: 1.day.ago)
|
|
||||||
assert_no_difference -> { users(:bob).readouts.count } do
|
|
||||||
delete measurement_path(readout), as: :turbo_stream
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
test "update changes readout value" do
|
|
||||||
readout = @user.readouts.create!(quantity: @quantity, unit: @unit, value: 82.5, taken_at: 1.day.ago)
|
|
||||||
patch measurement_path(readout), params: {
|
|
||||||
readout: { value: '83.0', unit_id: @unit.id, taken_at: readout.taken_at.iso8601 }
|
|
||||||
}, as: :turbo_stream
|
|
||||||
assert_response :success
|
|
||||||
assert_in_delta 83.0, readout.reload.value
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -1,64 +0,0 @@
|
|||||||
require "application_system_test_case"
|
|
||||||
|
|
||||||
class MeasurementsTest < ApplicationSystemTestCase
|
|
||||||
setup do
|
|
||||||
@user = sign_in(user: users(:alice))
|
|
||||||
|
|
||||||
@quantity = @user.quantities.create!(name: 'Weight')
|
|
||||||
@unit = @user.units.create!(symbol: 'kg')
|
|
||||||
@readout = @user.readouts.create!(quantity: @quantity, unit: @unit, value: 82.5)
|
|
||||||
|
|
||||||
visit measurements_path
|
|
||||||
end
|
|
||||||
|
|
||||||
test "index shows quantity name as edit link for active user" do
|
|
||||||
within 'tbody' do
|
|
||||||
assert_selector :link, exact_text: @quantity.name
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
test "edit opens inline form on quantity link click" do
|
|
||||||
within 'tbody' do
|
|
||||||
click_on @quantity.name
|
|
||||||
assert_selector ':focus'
|
|
||||||
assert_selector 'input[name="readout[value]"]'
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
test "edit and update measurement value" do
|
|
||||||
within 'tbody' do
|
|
||||||
click_on @quantity.name
|
|
||||||
fill_in 'readout[value]', with: '83.1'
|
|
||||||
assert_difference ->{ @readout.reload.value }, 83.1 - @readout.value do
|
|
||||||
click_on t('helpers.submit.update')
|
|
||||||
end
|
|
||||||
assert_no_selector :fillable_field
|
|
||||||
assert_selector :link, exact_text: @quantity.name
|
|
||||||
end
|
|
||||||
assert_selector '.flash.notice', text: t('measurements.update.success')
|
|
||||||
end
|
|
||||||
|
|
||||||
test "cancel edit restores original row" do
|
|
||||||
within 'tbody' do
|
|
||||||
click_on @quantity.name
|
|
||||||
assert_selector 'input[name="readout[value]"]'
|
|
||||||
click_on t(:cancel)
|
|
||||||
assert_no_selector :fillable_field
|
|
||||||
assert_selector :link, exact_text: @quantity.name
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
test "wide view edit opens panel form" do
|
|
||||||
@readout.update!(taken_at: Time.now)
|
|
||||||
visit measurements_path
|
|
||||||
execute_script("localStorage.removeItem('measurements-view')")
|
|
||||||
visit measurements_path
|
|
||||||
|
|
||||||
find('button[data-view="wide"]').click
|
|
||||||
within '#measurements-wide' do
|
|
||||||
assert_text format("%.10g", 82.5), wait: 3
|
|
||||||
find('button.link').click
|
|
||||||
end
|
|
||||||
assert_selector '#measurement_edit_form input[name="readout[value]"]', wait: 5
|
|
||||||
end
|
|
||||||
end
|
|
||||||
@@ -1,45 +0,0 @@
|
|||||||
require "application_system_test_case"
|
|
||||||
|
|
||||||
class QuantitiesTest < ApplicationSystemTestCase
|
|
||||||
setup do
|
|
||||||
@user = sign_in(user: users(:alice))
|
|
||||||
@unit = @user.units.create!(symbol: 'kg')
|
|
||||||
@quantity = @user.quantities.create!(name: 'Weight')
|
|
||||||
visit quantities_path
|
|
||||||
end
|
|
||||||
|
|
||||||
test "update button turns red when default unit changes" do
|
|
||||||
click_on 'Weight'
|
|
||||||
|
|
||||||
button = find('button[name=button]')
|
|
||||||
initial_color = evaluate_script("getComputedStyle(arguments[0]).backgroundColor", button)
|
|
||||||
|
|
||||||
select 'kg', from: 'quantity[default_unit_id]'
|
|
||||||
|
|
||||||
changed_color = evaluate_script("getComputedStyle(arguments[0]).backgroundColor", button)
|
|
||||||
refute_equal initial_color, changed_color, "Button color should change when default unit is altered"
|
|
||||||
end
|
|
||||||
|
|
||||||
test "saving default unit pre-selects it in measurements form" do
|
|
||||||
click_on 'Weight'
|
|
||||||
select 'kg', from: 'quantity[default_unit_id]'
|
|
||||||
click_on t('helpers.submit.update')
|
|
||||||
assert_selector '.flash.notice'
|
|
||||||
|
|
||||||
@quantity.reload
|
|
||||||
assert_equal @unit.id, @quantity.default_unit_id
|
|
||||||
|
|
||||||
visit measurements_path
|
|
||||||
find(:link_or_button, t('measurements.index.new_measurement')).click
|
|
||||||
assert_selector '#measurement_form'
|
|
||||||
|
|
||||||
within '#quantity_select' do
|
|
||||||
check 'Weight'
|
|
||||||
end
|
|
||||||
find('button[formaction]').click
|
|
||||||
|
|
||||||
within 'tbody#readouts' do
|
|
||||||
assert_selector "option[value='#{@unit.id}'][selected]"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
@@ -229,7 +229,7 @@ class UsersTest < ApplicationSystemTestCase
|
|||||||
user = User.find_by_email!(first(:link).text)
|
user = User.find_by_email!(first(:link).text)
|
||||||
inject_button_to first('td:not(.link)'), "update status", user_path(user), method: :patch,
|
inject_button_to first('td:not(.link)'), "update status", user_path(user), method: :patch,
|
||||||
params: {user: {status: User.statuses.keys.sample}}, data: {turbo: false}
|
params: {user: {status: User.statuses.keys.sample}}, data: {turbo: false}
|
||||||
execute_script("arguments[0].click()", find_button("update status"))
|
click_on "update status"
|
||||||
end
|
end
|
||||||
assert_title 'The change you wanted was rejected (422)'
|
assert_title 'The change you wanted was rejected (422)'
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user