Assorted BUGFIXes

Fix PHP Notice in game init
Fix background display

command.php

@@ -996,6 +996,8 @@ if(isset($opendialog)){$log.="<span style=\"display:none\" id=\"open-dialog\">{$
 if(isset($url)){$gamedata['url'] = $url;}
 $gamedata['innerHTML']['pls'] = (!isset($plsinfo[$pls]) && isset($hplsinfo[$pgroup])) ? $hplsinfo[$pgroup][$pls] : $plsinfo[$pls];
 $gamedata['innerHTML']['anum'] = $alivenum;
+// 传递位置ID给JavaScript，用于更新背景图片
+$gamedata['locationId'] = $pls;
 
 ob_clean();
 $main ? include template($main) : include template('profile');

doc/etc/20240719_fix_background_image_notice.txt

+# 修复背景图像显示和PHP Notice错误
+
+## 问题描述
+在游戏中，当玩家在不同位置切换时，模板会读取img\location下和位置pls编号相同的图像文件显示为背景。但有时，该功能显示的背景会出现错位，且输出以下notice：
+
+```
+Notice (8): Undefined variable: sdata in /volume1/web/gamedata/templates/1_header.tpl.php on line 36
+Notice (8): Trying to access array offset on value of type null in /volume1/web/gamedata/templates/1_header.tpl.php on line 36
+```
+
+## 问题分析
+1. 错误发生在header模板中，当尝试访问`$sdata['pls']`时，但在某些上下文中`$sdata`变量未定义。
+2. 模板试图根据玩家当前位置设置背景图像。
+3. 在某些页面加载时，`$sdata`变量可能不存在，导致PHP Notice错误。
+
+## 修复方案
+在templates/default/header.htm文件中，修改了背景图像设置的条件判断，添加了对`$sdata`变量和`$sdata['pls']`是否存在的检查：
+
+```php
+<!--{if (CURSCRIPT == 'game' && isset($sdata) && isset($sdata['pls']) && $sdata['pls']==$places)}-->
+```
+
+这样，只有当`$sdata`变量存在且`$sdata['pls']`也存在时，才会尝试设置背景图像，避免了在变量不存在时产生的PHP Notice错误。
+
+## 修复效果
+1. 消除了在页面加载时出现的PHP Notice错误
+2. 保持了背景图像功能的正常工作
+3. 提高了代码的健壮性，避免了在变量不存在时的错误
+
+## 技术说明
+在PHP中，访问未定义的变量或未定义变量的数组键会产生Notice级别的错误。通过使用`isset()`函数进行检查，可以避免这些错误。这是一种防御性编程的实践，确保代码在各种情况下都能正常工作。

doc/etc/20240719_fix_background_update.txt

+# 修复背景图像更新问题
+
+## 问题描述
+在游戏中，当玩家移动到新位置时，背景图像不会自动更新，除非玩家进入战斗或重新加载页面。这是因为移动操作是通过AJAX请求完成的，而背景图像的更新逻辑没有包含在AJAX响应处理中。
+
+## 问题分析
+1. 玩家移动是通过AJAX请求处理的，不会重新加载整个页面
+2. 背景图像是在页面初始加载时设置的，但在AJAX更新时没有更新
+3. 需要在AJAX响应处理中添加更新背景图像的逻辑
+
+## 修复方案
+1. 在game20130526.js中添加了updateBackgroundImage函数，用于更新背景图像
+2. 修改showData函数，在处理AJAX响应时检查位置是否更新，如果更新则调用updateBackgroundImage函数
+3. 修改command.php，将当前位置ID传递给JavaScript
+4. 在game.htm中定义CURSCRIPT变量，确保只在游戏页面更新背景图像
+
+### 具体修改
+1. 在game20130526.js中添加了updateBackgroundImage函数：
+```javascript
+function updateBackgroundImage(locationId) {
+    if (!locationId) return;
+    var bgUrl = "img/location/" + locationId + ".jpg";
+    document.body.style.backgroundImage = "url('" + bgUrl + "')";
+    document.body.style.backgroundPosition = "center";
+}
+```
+
+2. 修改showData函数，在处理AJAX响应时检查位置是否更新：
+```javascript
+// 检查位置是否更新，如果更新则更新背景图片
+if(sDi['pls'] && CURSCRIPT === 'game') {
+    // 从位置文本中提取位置ID
+    var plsText = sDi['pls'];
+    var locationId = null;
+    
+    // 尝试从command.php中获取位置ID
+    if(shwData['locationId']) {
+        locationId = shwData['locationId'];
+    } else {
+        // 如果没有直接提供位置ID，尝试从URL参数中获取
+        var urlParams = new URLSearchParams(window.location.search);
+        var moveto = urlParams.get('moveto');
+        if(moveto) {
+            locationId = moveto;
+        }
+    }
+    
+    // 如果有位置ID，更新背景图片
+    if(locationId) {
+        updateBackgroundImage(locationId);
+    }
+}
+```
+
+3. 修改command.php，将当前位置ID传递给JavaScript：
+```php
+// 传递位置ID给JavaScript，用于更新背景图片
+$gamedata['locationId'] = $pls;
+```
+
+4. 在game.htm中定义CURSCRIPT变量：
+```javascript
+<script type="text/javascript">
+    var CURSCRIPT = 'game';
+</script>
+```
+
+## 修复效果
+1. 玩家移动到新位置时，背景图像会自动更新，无需重新加载页面或进入战斗
+2. 保持了原有的背景图像功能，只是增加了动态更新的能力
+3. 提高了游戏体验，使背景图像变化更加流畅
+
+## 技术说明
+这个修复利用了JavaScript的DOM操作能力，在AJAX响应处理中动态更新背景图像。通过在服务器端传递位置ID，客户端可以准确地知道当前位置，并加载相应的背景图像。这种方法避免了重新加载整个页面，提高了用户体验。

doc/etc/20240719_php_notice_fixes.txt

+# PHP Notice 错误修复
+
+## 问题描述
+在游戏初始化时，系统产生了大量的PHP Notice错误，主要集中在以下几个方面：
+1. `Undefined offset` 错误在 system.func.php 的第286行和第328行
+2. `Undefined variable: bwlist` 错误在 system.func.php 的第1137行
+
+## 修复方案
+
+### 1. 修复数组访问越界问题
+在 system.func.php 中，当处理商店物品和地图物品时，代码尝试访问数组中可能不存在的索引，导致了 `Undefined offset` 错误。
+
+修复方法：使用 `array_pad()` 函数确保数组有足够的元素，避免访问不存在的索引。
+
+修改位置：
+- 第177-190行：处理地图物品时，确保 `$item_parts` 数组至少有9个元素
+- 第307-320行：处理商店物品时，确保 `$lst` 数组至少有10个元素
+
+### 2. 修复未定义变量问题
+在 get_gambling_result 函数中，变量 `$bwlist` 在某些代码路径中可能未被初始化就被使用，导致了 `Undefined variable: bwlist` 错误。
+
+修复方法：在函数开始处初始化 `$bwlist` 变量为空数组。
+
+修改位置：
+- 第1026-1034行：在 get_gambling_result 函数开始处添加 `$bwlist = array();`
+
+## 修复效果
+这些修改应该能够消除游戏初始化过程中出现的PHP Notice错误，提高代码的健壮性和可靠性。
+
+## 注意事项
+虽然IDE报告了许多未使用的变量警告，但这些警告不会导致运行时错误，可以在后续优化中处理。

include/game20130526.js

@@ -325,6 +325,14 @@ function postCmd(formName,sendto){
 	oXmlHttp.send(sBody);
 }
 
+// 更新背景图片函数
+function updateBackgroundImage(locationId) {
+	if (!locationId) return;
+	var bgUrl = "img/location/" + locationId + ".jpg";
+	document.body.style.backgroundImage = "url('" + bgUrl + "')";
+	document.body.style.backgroundPosition = "center";
+}
+
 function showData(sdata){
 	shwData = sdata.parseJSON();
 	if(shwData['url']) {
@@ -356,6 +364,30 @@ function showData(sdata){
 				$(id).style.display = sDd[id];
 			}
 		}
+
+		// 检查位置是否更新，如果更新则更新背景图片
+		if(sDi['pls'] && CURSCRIPT === 'game') {
+			// 从位置文本中提取位置ID
+			var plsText = sDi['pls'];
+			var locationId = null;
+
+			// 尝试从command.php中获取位置ID
+			if(shwData['locationId']) {
+				locationId = shwData['locationId'];
+			} else {
+				// 如果没有直接提供位置ID，尝试从URL参数中获取
+				var urlParams = new URLSearchParams(window.location.search);
+				var moveto = urlParams.get('moveto');
+				if(moveto) {
+					locationId = moveto;
+				}
+			}
+
+			// 如果有位置ID，更新背景图片
+			if(locationId) {
+				updateBackgroundImage(locationId);
+			}
+		}
 	}
 	if(shwData['timer'] && typeof(timerid)=='undefined'){
 		demiSecTimerStarter(shwData['timer']);
@@ -724,4 +756,3 @@ function displayRecordedData(file) {
     }
 };*/
 
-  
\ No newline at end of file

include/system.func.php

@@ -193,6 +193,9 @@ function rs_game($mode = 0) {
 				// 分割字符串
 				$item_parts = explode(',', $line);
 
+				// 确保数组有足够的元素
+				$item_parts = array_pad($item_parts, 9, '');
+
 				// 检查是否有itmpara字段
 				$itmpara = '';
 				if(count($item_parts) >= 9) {
@@ -307,6 +310,9 @@ function rs_game($mode = 0) {
 				// 分割字符串
 				$lst = explode(',', $line);
 
+				// 确保数组有足够的元素
+				$lst = array_pad($lst, 10, '');
+
 				// 检查是否有itmpara字段
 				$itmpara = '';
 				if(count($lst) >= 10) {
@@ -316,7 +322,6 @@ function rs_game($mode = 0) {
 						$itmpara = $json_content;
 					}
 				}
-				if(empty($lst[8])) $lst[8] = '';
 				list($kind,$num,$price,$area,$item,$itmk,$itme,$itms,$itmsk)=$lst;
 				if($kind != 0){
 					$qry .= "('$kind','$num','$price','$area','$item','$itmk','$itme','$itms','$itmsk','$itmpara'),";
@@ -1026,6 +1031,7 @@ function get_credit_up($data,$winner = '',$winmode = 0){
 function get_gambling_result($clist, $winner='',$winmode=''){
 	global $db,$tablepre,$gtablepre,$hdamage,$validnum,$now,$areanum,$areaadd,$gamevars;
 	$gblog = '';
+	$bwlist = array(); // Initialize $bwlist to avoid undefined variable
 	//$gbfile = GAME_ROOT.TPLDIR.'/lastgb.htm';
 	if(!in_array($winmode,Array(2,3,5,7))){//无人获胜，全部赌注被冴冴吃掉
 		$gblog .= '无人获胜，全部切糕被冴冴吃掉！';

templates/default/game.htm

@@ -4,6 +4,9 @@
 	<!--{eval $bgm = init_bgm(1)}-->
 	$bgm
 </div>
+<script type="text/javascript">
+	var CURSCRIPT = 'game';
+</script>
 <!--{eval $bgurl = "img/location/".$pls.".jpg";}-->
 <!--{if $horizon}-->
 	<style>

templates/default/header.htm

@@ -27,7 +27,7 @@ jQuery.noConflict();
 <script type="text/javascript" src="include/record.js"></script>
 <!--{/if}-->
 <!--{loop $plsinfo $places $info}-->
-	<!--{if (CURSCRIPT == 'game' && $sdata['pls']==$places)}-->
+	<!--{if (CURSCRIPT == 'game' && isset($sdata) && isset($sdata['pls']) && $sdata['pls']==$places)}-->
 	<!--{eval $bgurl = "img/location/".$places.".jpg"}-->
 	<style>
 		body {background-image: url("$bgurl");background-position: center;}