developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 11 years ago

Closed 11 years ago

#1011 closed bug (fixed)

Spelling errors throughout the code

Reported by: tudorilisoi Owned by: bballizlife
Priority: lowest Milestone: Jelix 1.0.12
Component: jelix Version: 1.1.3
Severity: trivial Keywords: spelling, code
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Hi, I see a lot of occurences of the 'unknow' word, which should be correctly spelled as as 'unknown' The misspelled word is found in libs, controllers, responses, and maybe many more source files

Attachments (1)

ticket_1011-branch_10x.diff (67.2 KB) - added by bballizlife 11 years ago.
Here is a patch for 1.0.x branch. Should convert all "unknow" to "unknown"

Download all attachments as: .zip

Change History (9)

comment:1 Changed 11 years ago by bballizlife

  • Owner set to bballizlife
  • Status changed from new to assigned

I will look at theses errors. Thanks for the report.

comment:2 Changed 11 years ago by bballizlife

  • Keywords spelling added; spellind removed

comment:3 Changed 11 years ago by tudorilisoi

probably a global replace will resolve the issue, but there are(veri slight) chances that this would break the code I recently noticed also in 'en_US' locale file errors.UTF-8.properties. The word is misspelled just about everywhere, I've developed about 3 weeks with Jelix and encountered a lot of occurences

comment:4 Changed 11 years ago by tudorilisoi

Sorry, I'm trying to type fast and as a consequence I mistyped some words. This is unwanted irony, rest assured. ;)

comment:5 Changed 11 years ago by bballizlife

hey hey ;) Never mind you write with errors as you can report us thoses we made ;)

comment:6 Changed 11 years ago by laurentj

  • Milestone set to Jelix 1.0.12

"unknow" is used in many literal strings, but also in some locale keys. For 1.0.x and 1.1.x, I propose to fix this word only in literal string. Do not change the keys, to not break applications if it uses these locale keys. We change the keys only in the trunk.

Changed 11 years ago by bballizlife

Here is a patch for 1.0.x branch. Should convert all "unknow" to "unknown"

comment:7 Changed 11 years ago by laurentj

patch commited in the 1.0.x, adapted and commited into the 1.1.x branch and in the trunk.

We still have to change keys of locales in the trunk.

comment:8 Changed 11 years ago by laurentj

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.