Merge pull request #3 from PandaMystique/claude/explain-codebase-mljwgl8sbvguuo21-sEdSC

Séparer données et logique: centraliser l'accès aux données dans data.js

Author: PandaMystique

www/index.html

@@ -307,8 +307,9 @@ <h1>Dictionnaire <span>de Philosophie</span></h1>
 
 <!-- JavaScript modules (order matters: dependencies first) -->
 
-<!-- Core: storage, state, utilities -->
+<!-- Core: storage, data layer, state, utilities -->
 <script src="js/storage.js"></script>
+<script src="js/data.js"></script>
 <script src="js/state.js"></script>
 <script src="js/theme.js"></script>
 <script src="js/search.js"></script>

www/js/article-ui.js

@@ -48,12 +48,8 @@ function showRandomArticle() {
 }
 
 // ===== FEATURE: NOTES =====
-function getNote(id) { return articleNotes[id] || ''; }
-function saveNote(id, text) {
-  if (text.trim()) articleNotes[id] = text.trim();
-  else delete articleNotes[id];
-  PhiloDB.set('philo-notes', JSON.stringify(articleNotes));
-}
+function getNote(id) { return Data.getNote(id); }
+function saveNote(id, text) { Data.saveNote(id, text); }
 
 // ===== FEATURE: FICHE EXPRESS =====
 function buildFicheExpress(entry) {
@@ -219,8 +215,7 @@ function showStats() {
 
 // ===== FEATURE: SORT =====
 function setSortMode(mode) {
-  sortMode = mode;
-  PhiloDB.set('philo-sort', mode);
+  Data.setPref('sortMode', mode);
   renderEntryList();
 }
 
@@ -260,7 +255,7 @@ function mobileJumpToLetter(letter) {
 
 // ===== FEATURE: COLLECTIONS =====
 function saveCollections() {
-  PhiloDB.set('philo-collections', JSON.stringify(collections));
+  Data.saveCollections();
 }
 
 function addToCollection(colName, articleId) {
@@ -336,7 +331,7 @@ function stopReadingTimer() {
     if (elapsed > 5 && elapsed < 3600) { // Between 5s and 1h
       var id = readingStartTimes._currentId;
       readingStartTimes[id] = (readingStartTimes[id] || 0) + elapsed;
-      PhiloDB.set('philo-reading-times', JSON.stringify(readingStartTimes));
+      Data.saveReadingTimes();
     }
   }
   readingStartTimes._current = null;

www/js/article.js

@@ -494,7 +494,7 @@ function toggleBookmark() {
   const idx = bookmarks.indexOf(currentArticle.id);
   if (idx > -1) bookmarks.splice(idx, 1);
   else bookmarks.push(currentArticle.id);
-  PhiloDB.set('philo-bookmarks', JSON.stringify(bookmarks));
+  Data.saveBookmarks();
 
   if (isMobile()) { updateMobileBookmarkBtn(); }
   showArticle(currentArticle.id);

www/js/custom-tags.js

@@ -1,49 +1,38 @@
 // ===== CUSTOM TAGS =====
-var customArticleTags = JSON.parse(lsGet('philo-custom-tags', '{}'));
-var allCustomTagNames = JSON.parse(lsGet('philo-all-custom-tags', '[]'));
+// Data (customArticleTags, allCustomTagNames) is managed by Data layer via aliases in state.js
 
 function getCustomTags(id) {
-  return customArticleTags[id] || [];
+  return Data.getCustomTags()[id] || [];
 }
 
 function saveCustomTags() {
-  var json = JSON.stringify(customArticleTags);
-  PhiloDB.set('philo-custom-tags', json);
-  try { localStorage.setItem('philo-custom-tags', json); } catch(e) {}
-  var names = JSON.stringify(allCustomTagNames);
-  PhiloDB.set('philo-all-custom-tags', names);
-  try { localStorage.setItem('philo-all-custom-tags', names); } catch(e) {}
+  Data.saveCustomTags();
 }
 
 function addCustomTag(id, tagName) {
   tagName = tagName.trim().toLowerCase();
   if (!tagName || tagName.length > 30) return;
-  if (!customArticleTags[id]) customArticleTags[id] = [];
-  if (customArticleTags[id].indexOf(tagName) >= 0) return;
-  customArticleTags[id].push(tagName);
-  if (allCustomTagNames.indexOf(tagName) < 0) allCustomTagNames.push(tagName);
+  var tags = Data.getCustomTags();
+  var names = Data.getAllCustomTagNames();
+  if (!tags[id]) tags[id] = [];
+  if (tags[id].indexOf(tagName) >= 0) return;
+  tags[id].push(tagName);
+  if (names.indexOf(tagName) < 0) names.push(tagName);
   saveCustomTags();
   showArticle(id);
 }
 
 function removeCustomTag(id, tagName) {
-  if (!customArticleTags[id]) return;
-  customArticleTags[id] = customArticleTags[id].filter(function(t) { return t !== tagName; });
-  if (customArticleTags[id].length === 0) delete customArticleTags[id];
+  var tags = Data.getCustomTags();
+  if (!tags[id]) return;
+  tags[id] = tags[id].filter(function(t) { return t !== tagName; });
+  if (tags[id].length === 0) delete tags[id];
   saveCustomTags();
   showArticle(id);
 }
 
 function promptAddCustomTag(id) {
-  // Show inline prompt with suggestions
-  var existingHtml = allCustomTagNames.length > 0
-    ? '<div style="display:flex;flex-wrap:wrap;gap:0.25rem;margin-bottom:0.5rem;">' +
-      allCustomTagNames.slice(0, 15).map(function(t) {
-        return '<span class="custom-tag" onclick="addCustomTag(\'' + id + '\',\'' + t.replace(/'/g, "\\'") + '\')">' + t + '</span>';
-      }).join('') + '</div>'
-    : '';
-  
-  var name = prompt('Nouvelle étiquette :');
+  var name = prompt('Nouvelle \u00e9tiquette :');
   if (name) addCustomTag(id, name);
 }
 
@@ -54,7 +43,7 @@ function buildCustomTagsHtml(id) {
     html += '<span class="custom-tag">' + tag +
       ' <span class="custom-tag-remove" onclick="event.stopPropagation();removeCustomTag(\'' + id + '\',\'' + tag.replace(/'/g, "\\'") + '\')">&times;</span></span>';
   });
-  html += '<button class="custom-tag-add" onclick="promptAddCustomTag(\'' + id + '\')" title="Ajouter une étiquette">+</button>';
+  html += '<button class="custom-tag-add" onclick="promptAddCustomTag(\'' + id + '\')" title="Ajouter une \u00e9tiquette">+</button>';
   html += '</div>';
   return html;
 }