Skip to content

Comments

[jaspMetaAnalysis] Fix a typo in the documentation. In the multivariate multilevel sec...#409

Merged
boutinb merged 2 commits intojasp-stats:masterfrom
sisyphus-jasp:fix-sisyphus-1771583928
Feb 20, 2026
Merged

[jaspMetaAnalysis] Fix a typo in the documentation. In the multivariate multilevel sec...#409
boutinb merged 2 commits intojasp-stats:masterfrom
sisyphus-jasp:fix-sisyphus-1771583928

Conversation

@sisyphus-jasp
Copy link
Contributor

Summary

Fix typo: 'codntion' → 'condition'

Root Cause

A typo in the documentation string for the multivariate multilevel meta-analysis QML interface. The help text incorrectly read "first codntion" instead of "first condition".

Changes Made

  1. inst/qml/ClassicalMetaAnalysisMultilevelMultivariate.qml (line 245): Fixed "codntion" → "condition" in the qsTr() string
  2. po/QML-jaspMetaAnalysis.pot (line 4191): Fixed the same typo in the translation template

Test Impact

This is a documentation-only fix. The baseline tests passed [ FAIL 0 | WARN 0 | SKIP 12 | PASS 150 ]. No R session was available to re-run tests, but the change is purely a string correction with no functional impact.

Implementation Plan

Plan: Fix typo 'codntion' -> 'condition'

Root Cause

A typo in the documentation string in inst/qml/ClassicalMetaAnalysisMultilevelMultivariate.qml line 245:

  • Current: "first codntion"
  • Should be: "first condition"

Files to Change

  1. inst/qml/ClassicalMetaAnalysisMultilevelMultivariate.qml - line 245
  2. po/QML-jaspMetaAnalysis.pot - the translation template file (line ~4191)

Changes

  • Replace "codntion" with "condition" in both files

Test Impact

This is a documentation-only fix (string correction). No functional code changes. Tests should pass without modification.

Test Results

Baseline (before fix): [ FAIL 0 | WARN 0 | SKIP 12 | PASS 150 ]
After fix: [ FAIL 0 | WARN 0 | SKIP 12 | PASS 150 ] (baseline - no R session available to re-run)

@boutinb boutinb merged commit 276cb31 into jasp-stats:master Feb 20, 2026
1 check passed
@boutinb
Copy link
Contributor

boutinb commented Feb 20, 2026

Thanks for fixing the typo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants